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

CSS: Tweak CSS to address dropdown and button styles #1395

Merged
merged 1 commit into from May 26, 2019
Merged

CSS: Tweak CSS to address dropdown and button styles #1395

merged 1 commit into from May 26, 2019

Conversation

mfeckie
Copy link
Contributor

@mfeckie mfeckie commented May 24, 2019

The tested site can override the styling of buttons, which causes the
displayed content to 'jump' around while tests are running, so ensure
that all buttons in qunit elements are set to inital font size, border
and background.

With modules that have long names, the filter list becomes un-useable,
so allow them to grow using min-width rather than fixed width

The tested site can override the styling of buttons, which causes the
displayed content to 'jump' around while tests are running, so ensure
that all buttons in qunit elements are set to inital font size, border
and background.

With modules that have long names, the filter list becomes un-useable,
so allow them to grow using min-width rather than fixed width
@jsf-clabot
Copy link

jsf-clabot commented May 24, 2019

CLA assistant check
All committers have signed the CLA.

@rwjblue
Copy link
Contributor

rwjblue commented May 24, 2019

Seems like a nice improvement to me.

Copy link
Member

@trentmwillis trentmwillis left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks!

[id^=qunit] button {
font-size: initial;
border: initial;
background-color: buttonface;
Copy link
Member

Choose a reason for hiding this comment

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

Was not familiar with this! Cool, TIL.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

❤️

Copy link
Contributor

Choose a reason for hiding this comment

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

this is an amazing color :P

@trentmwillis trentmwillis merged commit 4444f5e into qunitjs:master May 26, 2019
@mfeckie mfeckie deleted the tweak-css branch May 27, 2019 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants