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

Ensure that test labels wrap to fit within the container #119

Merged
merged 2 commits into from
Sep 11, 2013
Merged

Ensure that test labels wrap to fit within the container #119

merged 2 commits into from
Sep 11, 2013

Conversation

rviscomi
Copy link
Contributor

Test labels are commonly defaulted to the test URL. URL strings may not contain many word-breaking characters, so the text overflows beyond the container.

word-wrap: break-word; ensures that the text breaks as necessary to keep the entire label visible.

Read more: https://developer.mozilla.org/en-US/docs/Web/CSS/word-wrap

…ner.

Test labels are commonly defaulted to the test URL. URL strings may not contain many word-breaking characters, so the text overflows beyond the container.

word-wrap: break-word; ensures that the text breaks as necessary to keep the entire label visible.

Read more: https://developer.mozilla.org/en-US/docs/Web/CSS/word-wrap
Test labels are clickable on the comparison page when the results are in, but not clickable while the tests are still being run. Labels should always link to the test results page (or to a particular run, when available).

Need to override the color of the anchor from black to white for legibility.
@rviscomi
Copy link
Contributor Author

Also added commit to make test labels clickable while tests are running. Will use separate pull requests in the future.

pmeenan pushed a commit that referenced this pull request Sep 11, 2013
Ensure that test labels wrap to fit within the container
@pmeenan pmeenan merged commit f46a69b into catchpoint:master Sep 11, 2013
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