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

Too many columns can not view all columns #434

Closed
mingchen opened this issue Mar 14, 2017 · 12 comments
Closed

Too many columns can not view all columns #434

mingchen opened this issue Mar 14, 2017 · 12 comments
Labels

Comments

@mingchen
Copy link

In list view, if I define a lot columns need to be shown, if the row length overflow the right of page, some columns can not be viewed and the page also can not support horizontal scroll.
For example, I define 12 columns, however only 10 can be viewed on page, 2 of them are in right of screen and can not be viewed.

@wadjeroudi
Copy link
Contributor

@fzaninotto
Copy link
Member

Fixed in #57 (currently only in the next branch)

https://github.com/marmelab/admin-on-rest/pull/57/files#diff-8ff019c2b429722635e5d0ce58e2c0e2

@mingchen
Copy link
Author

It is still reproducible in the latest code.

To reproduce, open following page:

https://marmelab.com/admin-on-rest-demo/#/commands?_k=wi0hso

Then change window size to make it small in width, you will see some part of right table can not be viewed and it can not horizontal scroll.

@fzaninotto
Copy link
Member

Again, this is fixed on the next branch, not yet released.

@1corner
Copy link

1corner commented Oct 23, 2017

We still experience this issue in the latest release. I also tested https://marmelab.com/admin-on-rest-demo/#/commands and have the same issue there. Both on iPhone Safari and Chrome but also when resizing on chrome on desktop. The horizontal scroll do not appear so I can not see columns or edit button.

@data-server
Copy link

@fzaninotto this issue has been fixed? can you give me some link

@djhi
Copy link
Collaborator

djhi commented Feb 7, 2018

The next branch has been released. However the project is now named react-admin:

npm install [email protected].

Be sure to check the upgrade guide in the next branch too

@data-server
Copy link

@djhi has upgraded , but the issue hasn't been fixed

djhi added a commit that referenced this issue Feb 7, 2018
@fzaninotto
Copy link
Member

Re-fixed by #1497

@data-server
Copy link

@fzaninotto next branch has fixed this issue, but master isn't fixed yet.

@fzaninotto
Copy link
Member

Reopening to consider the fix in master

@fzaninotto
Copy link
Member

Moved to marmelab/admin-on-rest#2.

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

6 participants