-
-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
improved footer layout #1527
improved footer layout #1527
Conversation
Agree the footer could be improved. This change breaks the topside theme. Not sure why Also try replacing the revid with rev_number |
I like this text because you can use it to search the web to find all running Moin wikis. Maybe we should change it to "MoinMoin2 powered" or similar to distinguish between old and new versions. |
The |
I don't think so because at https://moinmo.in/ you can find information about both Moin 1.9 and Moin2 |
topside theme seems broken for small screen sizes |
Is this so bad? Sometimes the history has more items and than it needs more space. In this case it seems better if the history items begin on the left side. But we could align it so it is on the same height as other elements:
Okay, I reversed most of the layout changes and did a commit but the icons of the item views still take more place than before. The reason for this is that on a small device it is harder to tap on small icons so we prevent tapping on the wrong icon:
Of course we can change the image in this PR :) It would look like this: Or if we take the bottom of that picture: Almost invisible but you can still see the color gradient. |
In your last screenshots there is a line break in "HTML 5", maybe we should change it to "HTLM5" in templates/snippets.html. |
That's correct but it happens only if the device has a smaller width than 380px which is really small ;) But others like Mozilla or Wikipedia are also calling it 'HTML5' so it makes sense to change it so I did it. What do you say about the suggestions for the image of the footer? Do you prefer the first or the second one (third and forth image in #1527 (comment))? |
I prefer the 4th image (the bottom of white-clouds.jpg), but both are fine. If your work is finished, may I ask you to create a new short PR with one commit rebased on the latest version? |
43982fe
to
12dd678
Compare
12dd678
to
5cb8c2d
Compare
Of course :) I changed the background picture of the footer and then I saw that I forgot to implement the change in the header (see #1527 (comment)) so I just worked on that. Please check if the changes of the modern theme on mobile are okay now. In the end I rebased the branch on your |
IMO this is ready to merge. @RogerHaase do you agree? |
I like the footer background. Using modernized theme, there is a horizontal scroll bar at window widths below ~850 pixels. Using topside theme, there is a horizontal scroll bar at ~1200 pixels wide. The icons for moin-powered and python-powered shift from right to left-center at windows widths from ~850 to ~1000. |
@RogerHaase, this should look better now. |
Thanks for helping! |
Improves the layout of the footer:
Before:
After