Bug 7955 - Installer and setup page margins are duplicated on each page
Summary: Installer and setup page margins are duplicated on each page
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Server Installer (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: LowPrio
Assignee: Bugzilla mail exporter
URL:
Keywords:
Depends on: 7953
Blocks:
  Show dependency treegraph
 
Reported: 2022-06-28 09:27 CEST by William Sjöblom
Modified: 2022-06-28 15:45 CEST (History)
0 users

See Also:
Acceptance Criteria:


Attachments

Description William Sjöblom cendio 2022-06-28 09:27:30 CEST
Each page in tl-setup and tlinstaller is currently responsible for
applying proper margins to the page contents. In a perfect world, the
margins should be applied centrally by the wizard itself to avoid
duplication.

As we use factory methods for applying the correct margins, things could
definitely be worse. Still, getting rid of the duplicated factory function
invocations would still be beneficial.
Comment 1 William Sjöblom cendio 2022-06-28 09:30:38 CEST
This issue is seemingly coupled with bug 7953. This coupling is mainly due
to some wizard pages consisting of notebooks with only some notebook
pages being scrollable. What I see as the neatest and least convoluted
approach to this is letting the whole nested notebook be
scrollable. This, however, requires at least some foundational work on
bug 7953.

Other than this, centralizing the page margin application looks fairly
trivial.
Comment 2 William Sjöblom cendio 2022-06-28 09:38:14 CEST
There is also the alternative of placing a ScrolledWindow within the nested notebook pages that need scrolling. The problem with this is that the scrollbars for these ScrollableWindows would also get the global page margins and would not sit flush against the window border.

There is of course the possibility to create a global scrollbar that each nested scrollable notebook page connects to when shown and disconnects from when hidden. I did some playing around with this but my overall feeling is that this approach is pretty error-prone and straight-up ugly as it requires a bunch of duplicated callbacks.
Comment 3 William Sjöblom cendio 2022-06-28 15:45:10 CEST
I have a strong suspicion that adding animations to the installer and setup may also get troubled by the nested notebooks. It is also related to this bug in general. So let's add a See Also.

Note You need to log in before you can comment on or make changes to this bug.