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

Made autoSizeColumns() to work with cells that have padding #852

Closed

Conversation

mpetkov
Copy link

@mpetkov mpetkov commented Apr 21, 2016

My suggested fix for issue #812

… the user is using auto size columns functionality.
@ceolter
Copy link
Contributor

ceolter commented Apr 26, 2016

thanks. will take a look when i get time.

@arkon
Copy link

arkon commented May 12, 2016

Why not just get the container's clientWidth instead of its offsetWidth? Wouldn't that include the padding and eliminate the need for these calculations?

@ceolter
Copy link
Contributor

ceolter commented Jun 6, 2016

i went to work on this, and as far as i can work out, it's not needed.

in your example, i changed the overridden style to the following and it worked

.ag-fresh .ag-cell-not-inline-editing { padding: 0 40px; }

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