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
[api] Make image template page look amazing!! #2380
Conversation
Nice work |
word-break: break-all | ||
|
||
/* Non standard for webkit */ | ||
word-break: break-word |
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.
Doesn't this line overwrite the previous one?
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.
only if the browser understands it - i.e. webkit. All other browsers will ignore it as syntax error.
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.
Right! Only if the browser supports it.
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.
I see.
btw the indentation seems to be wrong
= link_to(title_or_name(template), package_show_path(project: project, package: template)) | ||
%span.description.grey | ||
%span.description.grey.break-words |
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.
and thumbs up for using a class and not an id for styling 👍
Style changes in the image template page: - Show more test from the name and description. - Do not hide any word of the name/description because it is too long. - `...` is shown when the name/description is too long (if the browser supports it). - Project name in bold. - Reduce top padding for title.
Style changes in the image template page:
...
is shown when the name/description is too long (if the browser supports it).Now it looks amazing!!
Before:
Now:
Chromium
Firefox