-
Notifications
You must be signed in to change notification settings - Fork 0
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
fix: fix responsiveness on mobile resolutions #2
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, note that commits of type style:
addresses changes such as formatting. Fixes to styling issues must have type fix:
@@ -91,6 +93,11 @@ public DayOfWeekSelector() { | |||
buttonsLayout.add(new DayOfWeekButton(DayOfWeek.FRIDAY, "F")); | |||
buttonsLayout.add(new DayOfWeekButton(DayOfWeek.SATURDAY, "S")); | |||
|
|||
buttonsLayout.setWidthFull(); | |||
buttonsLayout.setJustifyContentMode(JustifyContentMode.CENTER); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Style in CSS file (otherwise it cannot be overriden)
@@ -81,6 +82,7 @@ private void setState(boolean state) { | |||
*/ | |||
public DayOfWeekSelector() { | |||
getStyle().set("padding", "var(--lumo-space-m)"); | |||
getStyle().set("width", "100%"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace with setWidthFull();
e8e2ab2
to
4fc14b4
Compare
Close #3 |
4fc14b4
to
1ebc4d4
Compare
This PR will solve the following issue in ActivityLog: Fix responsiveness in TimestampFilterDialog #93