Skip to content

double click between column headers to autoresize does not take header size into consideration #43

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

Closed
gdementen opened this issue Aug 25, 2017 · 3 comments
Assignees
Labels
Milestone

Comments

@gdementen
Copy link
Collaborator

gdementen commented Aug 25, 2017

it only takes the data into consideration (while it should use both header and data), and if the header size is larger than the data, it is annoying, because the header text is not entirely visible.

@gdementen gdementen added the bug label Aug 25, 2017
@alixdamman alixdamman self-assigned this Aug 25, 2017
alixdamman added a commit to alixdamman/larray-editor that referenced this issue Aug 25, 2017
alixdamman added a commit to alixdamman/larray-editor that referenced this issue Aug 25, 2017
@gdementen
Copy link
Collaborator Author

somehow this problem is still present

@gdementen gdementen reopened this Sep 16, 2017
@gdementen gdementen added this to the 0.26 milestone Sep 18, 2017
@gdementen
Copy link
Collaborator Author

More precisely, it works for the first column(s) (the label columns), but not the data columns. If the header length is longer than the width for data cell, it is not shown entirely. For testing, we can use:

arr = zeros('a=a_long_label,another_long_label')

alixdamman added a commit to alixdamman/larray-editor that referenced this issue Sep 21, 2017
@gdementen
Copy link
Collaborator Author

Still not fixed :) For the "index" columns, it only take axes names into consideration, not labels.

arr = ndrange('a=long_long_long;b=b0,b1')

Also the axes "view" width is not reset when going from one array to another, which means it desync with the "vlabels" view, leading to odd results.

@gdementen gdementen reopened this Sep 26, 2017
alixdamman added a commit to alixdamman/larray-editor that referenced this issue Sep 27, 2017
…) and resizeAxesRowToContents() methods + synchronized view_axes with view_x/ylabels in ArrayEditorWidget
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants