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

Updating column width when columnDefs changes #3897

Merged
merged 1 commit into from Jul 1, 2015

Conversation

lpand
Copy link
Contributor

@lpand lpand commented Jun 30, 2015

This fixes issue #3880. The width of the column definition should have precedence over the current column width.

Maybe it'd be better to put that block of code into a separated method?

Review on Reviewable

@JLLeitschuh
Copy link
Contributor

Test failure:

PhantomJS 1.9.8 (Linux 0.0.0) ui.grid.resizeColumns clicking on a resizer and moving the mouse then releasing the mouse should cause the column to resize by the amount change in the X axis FAILED
    Expected 0 to equal 100.

@c0bra
Copy link
Contributor

c0bra commented Jun 30, 2015

@lpand a single commit is preferred for pull requests. You can squash yours and and force push it to your fork. Here's a guide on how to do it: https://github.com/ginatrapani/todo.txt-android/wiki/Squash-All-Commits-Related-to-a-Single-Issue-into-a-Single-Commit

…column definition

set column width to * if coldef's width is not a number or a suitable string

just refresh the canvas after a column is resized
@lpand
Copy link
Contributor Author

lpand commented Jun 30, 2015

First time i read about this feature of git... Nice.

c0bra added a commit that referenced this pull request Jul 1, 2015
Updating column width when columnDefs changes
@c0bra c0bra merged commit 4157771 into angular-ui:master Jul 1, 2015
@c0bra
Copy link
Contributor

c0bra commented Jul 1, 2015

LGTM. Thanks a lot, @lpand!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants