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

Preserve line breaks in project descriptions #1017

Merged
merged 1 commit into from
Dec 12, 2016
Merged

Preserve line breaks in project descriptions #1017

merged 1 commit into from
Dec 12, 2016

Conversation

rhamilto
Copy link
Member

@rhamilto rhamilto commented Dec 9, 2016

Resolves openshift/origin#12194

Limits line breaks to 10 for the truncated display (does not apply to the highlighted text display). Also adds word-break for long, unbroken strings so they do not break the layout.

screen shot 2016-12-09 at 1 55 28 pm
screen shot 2016-12-09 at 1 55 35 pm
screen shot 2016-12-09 at 1 55 45 pm
screen shot 2016-12-09 at 1 56 43 pm

@jwforres or @spadgett, PTAL

@jwforres
Copy link
Member

jwforres commented Dec 9, 2016

so in other places we chose not to show all text when highlighting because it made things jump around

@rhamilto
Copy link
Member Author

The show all text when highlighting is not a change to this PR. The existing word limit was/is 1000 characters. So I assumed that since we're already showing a max of 1000 characters (which seems like a whole lotta characters for a description), we probably don't won't to limit the line breaks, either. Otherwise, you could end up with a situation where the highlighting isn't visible. Yea/nay?

@jwforres
Copy link
Member

ah k thats fine then, maybe i misremembered the result of our previous discussion on showing more when filtering.

@jwforres
Copy link
Member

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 3182c70

@openshift-bot
Copy link

openshift-bot commented Dec 12, 2016

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/839/) (Base Commit: 8f964a8)

@openshift-bot openshift-bot merged commit 3e63c9c into openshift:master Dec 12, 2016
@rhamilto rhamilto deleted the issue-12194 branch December 12, 2016 14:46
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.

None yet

3 participants