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
Add proper styling to DlnaProfile and ScheduledTask pages #353
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.
@@ -139,8 +139,8 @@ | |||
} | |||
|
|||
.dialog-small { | |||
width: 60%; | |||
height: 80%; | |||
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.
Something seems a little bit weird here, .dialog-small
becomes bigger than .dialog-medium
. Maybe you should use another class or rename this one?
This page is already weird with all the bullet points, but disregarding that, I think 1 is more readable. |
@DrPandemic way u don't get the same rendering on firefox and chrome. b/c on firefox the emby-button inject button-inline class for this i will open a new PR that fix for both. And for the bullet points i will remove it only it appere on the title |
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.
Except for the button-link-inline
this looks good!
See PR #356 Fix some style |
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.
The changes look good to me, I can't believe DlnaProfile never used the style used everywhere else in the frontend.
Before
After