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

Pagination formatting is broken #73

Closed
matthiaswh opened this issue Oct 17, 2019 · 3 comments · Fixed by #77
Closed

Pagination formatting is broken #73

matthiaswh opened this issue Oct 17, 2019 · 3 comments · Fixed by #77
Assignees
Labels

Comments

@matthiaswh
Copy link
Contributor

66865768-8e4e3d80-ef5d-11e9-8b13-b52480f4bb90

@matthiaswh matthiaswh added the bug label Oct 17, 2019
@matthiaswh matthiaswh self-assigned this Oct 17, 2019
@guruofgentoo
Copy link
Member

FWIW, in an app the other day I made the styling around that area a bit more specific:

.datagrid .paging {
    margin-right: 0 !important;
}

.datagrid .paging th, .datagrid .sorting dt {
    font-size: 12px;
}

.datagrid .paging .perpage {
    width: 75px;
}

It looks like there is more happening here, though, with the overlapping, which may have to do with other style elements in the app template you're working with.

@matthiaswh
Copy link
Contributor Author

matthiaswh commented Oct 18, 2019

I noticed the overlapping also occurring on another project, which has basically no added styles. This makes it difficult to use the sort dropdown as most of its hitbox is covered by the absolute positioned table that should sit below it:

image

This is a quick fix for the overlapping:

.datagrid form.header {
  min-height: 140px;
}

But the layout for this whole section is pretty archaic, using floats and tables... that also makes it relatively inflexible and easy to break. I took a quick stab at modernizing it using flexbox, which has broad support back to IE11, albeit with a couple of the usual IE bugs. At the same time I attempted to improve the layout to make it more obvious that you need to use the Apply button for all of the elements.

image

Open to feedback, of course.

@guruofgentoo
Copy link
Member

I'm all for fixing the layout so it applies properly in all modern browsers and works with more generic templates. But, moving things around too much will be problematic for some of our larger apps, who have some power users who are used to the layout. IMO we should save broader facelifts for webgrid 2.0, which will have additional options/settings to place.

I'd be fine with having the Apply button in it's own "row", but I think it needs to stay on the left. The vast majority of operations users make in that area are the filters. I have thought it would look better if Sort By were next to its first box, rather than above it, but I'd keep it over to the right, above the pagination controls.

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 a pull request may close this issue.

2 participants