Page MenuHomePhorge

hide menubar when in rooms
AbandonedPublic

Authored by bohlender on Feb 16 2021, 1:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 26, 8:18 PM
Unknown Object (File)
Thu, Nov 16, 5:25 PM
Unknown Object (File)
Oct 28 2023, 3:39 PM
Unknown Object (File)
Oct 23 2023, 11:22 AM
Unknown Object (File)
Oct 22 2023, 9:07 PM
Unknown Object (File)
Oct 19 2023, 2:05 AM
Unknown Object (File)
Sep 29 2023, 1:03 PM
Unknown Object (File)
Sep 28 2023, 11:00 AM
Subscribers

Details

Reviewers
None
Group Reviewers
Restricted Project

Diff Detail

Repository
rK kolab
Branch
menubargone
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 33232
Build 13018: arc lint + arc unit

Event Timeline

bohlender created this revision.
bohlender added a reviewer: Restricted Project.Feb 16 2021, 2:07 PM

The idea was to hide the menu/footer inside the room, i.e. after joining in, but room setup screen should have the menu as usual. I also don't like making component/page specific exceptions in the Menu widget. I would probably do it with css. Also the code added in Room.vue asks for some deduplication, i.e. we need a method on the $root component that could be re-used as we have the same code in at least two places now. Not to mention that in this state some browser tests will fail (but I didn't check yet).

I think I could give it a try, it's not a big task.

@machniak
Is this what you want instead: https://git.kolab.org/D2281 ?
Isn't that also one component knowing about the other and manipulating it ?

Is this what you want instead: https://git.kolab.org/D2281 ?

More like it. I would probably do this with css.

Isn't that also one component knowing about the other and manipulating it ?

Yes, but it is a page component not a widget. So, better, imo.