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

set white-space of first column in jstree to normal #6770

Conversation

jrief
Copy link
Contributor

@jrief jrief commented Nov 19, 2019

Description

If the length of a page title is too long, it pushes the JS-Tree out of the visible window, creating a horizontal scroll bar at the bottom:

Screen Shot 2019-11-19 at 22 47 42

To prevent horizontal scrolling, we can use the proper white-space: normal property, so that this text gets wrapped:

Screen Shot 2019-11-19 at 22 46 37

Checklist

  • I have opened this pull request against develop
  • I have updated the CHANGELOG.rst
  • I have added or modified the tests when changing logic

@coveralls
Copy link

coveralls commented Nov 19, 2019

Coverage Status

Coverage remained the same at 78.215% when pulling d0bde35 on jrief:fixes/jstree-first-column-wrap-white-space into 877c5c2 on divio:release/3.7.x.

@jrief jrief changed the title set white of first column in jstree to normal set white-space of first column in jstree to normal Nov 20, 2019
jrief added 3 commits May 5, 2020 11:09
…first-column-wrap-white-space

# Conflicts:
#	CHANGELOG.rst
#	cms/static/cms/css/3.7.2/cms.pagetree.css
…first-column-wrap-white-space

# Conflicts:
#	CHANGELOG.rst
#	cms/static/cms/css/3.7.2/cms.pagetree.css
…om:jrief/django-cms into fixes/jstree-first-column-wrap-white-space

# Conflicts:
#	CHANGELOG.rst
@jrief
Copy link
Contributor Author

jrief commented May 5, 2020

Rebased for version 3.7.2 and a friendly reminder that some CMS installations really use up to 255 characters for the page title.

jrief added 2 commits May 5, 2020 11:25
…om:jrief/django-cms into fixes/jstree-first-column-wrap-white-space

# Conflicts:
#	CHANGELOG.rst
…om:jrief/django-cms into fixes/jstree-first-column-wrap-white-space
@yakky yakky changed the base branch from release/3.7.x to develop May 24, 2020 11:48
@yakky yakky changed the base branch from develop to release/3.7.x May 24, 2020 11:48
@yakky
Copy link
Member

yakky commented May 24, 2020

@jrief could you rebase this against develop? It will be cherrypicked to 3.7.x branch for release. As I'd really like this to be in 3.7.3 which is due soon

BTW: This has been needed for a long time! 🎉

@yakky yakky added this to the 3.7.x milestone May 24, 2020
@jrief jrief mentioned this pull request May 24, 2020
3 tasks
@jrief
Copy link
Contributor Author

jrief commented May 24, 2020

@yakky ported changes as #6856 to develop branch

@yakky
Copy link
Member

yakky commented May 24, 2020

@jrief 🎉 thanks
I'll merge that as the CI will end

@yakky
Copy link
Member

yakky commented May 24, 2020

@FinalAngel should we close this, right? you will port this to 3.7 directly from develop

@jrief
Copy link
Contributor Author

jrief commented May 24, 2020

I would keep both PRs separate. Reason is that the css file is compiled and minified, and hence merging presumably impossible.

@yakky yakky mentioned this pull request May 26, 2020
@FinalAngel FinalAngel closed this Sep 8, 2020
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

4 participants