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

Use flex display on dialog windows #2778

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Conversation

jevenski
Copy link
Contributor

@jevenski jevenski commented Nov 4, 2024

I can't vertically scroll the RSS Manager on mobile, so everything below the heading "RatioGroup" is invisible. This includes the RatioGroup selection box, the Channel heading and selection box, as well as the Reset, OK, and Cancel buttons.

Originally posted by @Airman8 in #2721 (comment)

This PR address this issue mentioned above. It's based on the previous PR (#2762), with a few minor adjustments added. The previous PR was planned for v5.2 in the beginning, but now it seems we'd better include this in the v5.1 stable release.

  • Use flex display for dialog contents, and enable overflowing on the Y-axis if content height exceeds the max-height of windows.
  • Define background color and border colors using CSS variables.
  • Remove unused CSS rules.

- Use flex display for dialog contents, and enable overflowing on
  the Y-axis if content height exceeds the max-height of windows.
- Define background color and border colors using CSS variables.
- Remove unused CSS rules.
@stickz stickz merged commit a1c7969 into Novik:master Nov 4, 2024
@jevenski jevenski deleted the flex-dialog branch November 4, 2024 23:00
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