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

CRM-19456 - avoid horizontal scrolling #9173

Merged
merged 1 commit into from
Oct 12, 2016

Conversation

jmcclelland
Copy link
Contributor

@jmcclelland jmcclelland commented Oct 4, 2016

When no-wrap is specified, pages such as the Manage Groups and the
Activities tab will require horizontal scrolling because the
descriptions won't wrap.


When no-wrap is specified, pages such as the Manage Groups and the
Activities tab will require horizontal scrolling because the
descriptions won't wrap.
Copy link
Contributor

@seamuslee001 seamuslee001 left a comment

Choose a reason for hiding this comment

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

Tested this and agree its a good change.

@michaelmcandrew
Copy link
Contributor

@jmcclelland this looks like a dup of #9145 / https://issues.civicrm.org/jira/browse/CRM-19245 which has already been merged.

@michaelmcandrew
Copy link
Contributor

@jmcclelland - wondering what you think of the alternate strategy here: https://github.com/civicrm/civicrm-core/pull/9145/files @tschuettler - any thoughts on this as an alternative to your fix?

@jmcclelland
Copy link
Contributor Author

I prefer the broader reach of my patch - since it is not just the manage group page - it also affects the activities tab and probably more places.

@michaelmcandrew michaelmcandrew merged commit 4240e48 into civicrm:master Oct 12, 2016
@michaelmcandrew
Copy link
Contributor

michaelmcandrew commented Oct 12, 2016

@colemanw @jmcclelland - same here. I'm merging this in and going to revert 9145 here #9220

michaelmcandrew added a commit that referenced this pull request Oct 12, 2016
f2boot pushed a commit to f2boot/civicrm-core that referenced this pull request Oct 12, 2016
@tschuettler
Copy link
Contributor

@michaelmcandrew looks good to me.
You may want to consider that @colemanw did ask for a more specific fix: #8952

JohnFF pushed a commit to JohnFF/civicrm-core that referenced this pull request Oct 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants