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

fix: fix responsiveness on mobile resolutions #2

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

Valenreynoso17
Copy link
Contributor

This PR will solve the following issue in ActivityLog: Fix responsiveness in TimestampFilterDialog #93

Copy link
Member

@javier-godoy javier-godoy left a 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);
Copy link
Member

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%");
Copy link
Member

Choose a reason for hiding this comment

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

Replace with setWidthFull();

@javier-godoy javier-godoy changed the title style: fix responsiveness on mobile resolutions fix: fix responsiveness on mobile resolutions Jan 25, 2024
@javier-godoy
Copy link
Member

Close #3

@javier-godoy javier-godoy merged commit a668782 into master Jan 26, 2024
2 checks passed
@javier-godoy javier-godoy deleted the fix-responsiveness branch January 26, 2024 12:30
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