Skip to content
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

Display current time on same row as navigation buttons on narrow screens #171

Merged
merged 2 commits into from
Jan 7, 2022

Conversation

mstock
Copy link

@mstock mstock commented Dec 28, 2021

This moves the markup for the current time display inside the same element as the navigation buttons, which prevents it from being moved to a separate row on narrow displays, which in turn resulted in 'hidden' page elements as was noted in e.g. #141 (for DiVOC R2R) and IRC (for rC3 2021).

That way, on narrow displays, it should appear on the same row as the
buttons and not overlap the stream name/title anymore. Also use the
`navbar-text` class on the text for positioning.
This fixes the positioning of the buttons now that the time display is
inside the wrapper, and maybe makes a little more sense, too, since this
is not a form.
Copy link
Member

@iSchluff iSchluff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks

@iSchluff iSchluff closed this Jan 7, 2022
@iSchluff iSchluff reopened this Jan 7, 2022
@iSchluff iSchluff merged commit 5ab7583 into voc:master Jan 7, 2022
@mstock
Copy link
Author

mstock commented Jan 7, 2022

Great, thanks for merging! Since this should also resolve #141, you might consider closing that, too. (Altough the divoc_r2r CSS likely had the additional issue of missing something like

@media (max-width: 498px) { /* when the navbar breaks */
padding: 165px 0;
}
on the body element so the room name would still be hidden - but divoc_r2r is over and archived anyway ;).)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants