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

Prevent tags from getting cut by long build numbers #1321

Merged
merged 1 commit into from
May 16, 2017

Conversation

krauselukas
Copy link
Contributor

Related to https://progress.opensuse.org/issues/12942

Already fixed the issue with those long build numbers by shorten them with dots. But then the tags got cut aswell. Now the tags are always visible and just the text and timeago part gets shortend.

screenshot_2017-05-11_15-50-23

}

.col-lg-4.text-nowrap span {
font-size: 17px;
Copy link
Contributor

Choose a reason for hiding this comment

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

if this is to align with the font size of h4, better use $font-size-h4 - that's the reason we use sass

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Allright!

@codecov
Copy link

codecov bot commented May 12, 2017

Codecov Report

Merging #1321 into master will decrease coverage by 15.49%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1321      +/-   ##
==========================================
- Coverage   87.15%   71.65%   -15.5%     
==========================================
  Files         104       95       -9     
  Lines        7363     6968     -395     
==========================================
- Hits         6417     4993    -1424     
- Misses        946     1975    +1029
Impacted Files Coverage Δ
lib/db_profiler.pm 12% <0%> (-88%) ⬇️
lib/OpenQA/Scheduler/Locks.pm 7.46% <0%> (-85.08%) ⬇️
lib/OpenQA/Schema/Result/JobGroups.pm 27.41% <0%> (-72.59%) ⬇️
lib/OpenQA/Schema/Result/Assets.pm 9.35% <0%> (-72.52%) ⬇️
lib/OpenQA/Scheduler/Scheduler.pm 28.46% <0%> (-66.93%) ⬇️
lib/OpenQA/Schema/Result/Screenshots.pm 7.69% <0%> (-66.67%) ⬇️
lib/OpenQA/Schema/ResultSet/JobSettings.pm 13.33% <0%> (-60%) ⬇️
lib/OpenQA/WebAPI/Controller/API/V1/JobGroup.pm 47.05% <0%> (-50%) ⬇️
lib/db_helpers.pm 50% <0%> (-50%) ⬇️
lib/OpenQA/WebAPI/Plugin/Gru.pm 41.93% <0%> (-40.33%) ⬇️
... and 39 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f582711...67a0256. Read the comment docs.

@krauselukas
Copy link
Contributor Author

Fixed font-size part @coolo

@krauselukas
Copy link
Contributor Author

Had to import the theme_variables.scss file to the openqa.scss. Hope that is the right way. @coolo

@okurz
Copy link
Member

okurz commented May 15, 2017

please write commit message subject lines without the . at the end of line.

@krauselukas krauselukas changed the title Prevent tags from getting cut by long build numbers. Prevent tags from getting cut by long build numbers May 15, 2017
@coolo coolo merged commit 9e10ee0 into os-autoinst:master May 16, 2017
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.

3 participants