-
Notifications
You must be signed in to change notification settings - Fork 108
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
Column filter going on second row #554
Comments
Related #514 |
You can tweak the icon placement by changing the css class: atk-table-dropdown By default it is meant to float right, which will cause icon to go on next row if not enough room.
|
Hi ibelar, I take a look and didn't find how and where I can add my custom css or even how can I change this css class? |
i think it should display properly by default. We don't have many CSS developers, i'll assign to @mayack, but he's busy working on other things. |
In order to override the style, you would normally create your own css file and include the new css file in your app using App::requireCSS method.
Just redefine the rule for "atk-table.dropdown" in this new css file. Hope this help. |
Hi, a solution has been given to this problem and I don't know if it still occurs. #514 is already closed. I suggest we close this issue. |
Not sure about closing this. Filter (or sorting) icons should never go on next line in my opinion. |
This is a css issue. The css class name .atk-table-dropdown can be modify to fit everyone needs. |
@ibelar Why can't we set this CSS as default to look nice out-of-the-box? |
@DarkSide666 - The best compromise is to have the dropdown trigger icon on the next line, under the column label name by default. This way, it will always remain the same, independently of the column width. We cannot take into account that there will be enough space in each column to properly fit the sorting icon and the filtering icon on the same line. If we set CSS to have all icons on the same line by default, then, when the column width is changed and becomes not wide enough, (see picture below) we will end up with the filter icon placed on the next line anyway. I believe that it is the developer responsibility to try placing it on the same line and live with the consequence. Only they will know what type of table they have to deal with. Plus, new CSS class introduced in PR #900 should give more freedom to adjust this to their needs. |
How about placing the filter icon in front of the column label and opening the popup on click of the icon and the label text? |
@ibelar can this be solved with modern |
@mvorisek - I am not a CSS guru, so maybe. Guys, for me, this issue is closed. It might not be the prettiest solution but it is a workable one. There are workarounds to adapt to your needs, like changing CSS or using Table hook in order to generate your own column header. |
If Grid column content is shorter than column name, filter icon go on the next row

The text was updated successfully, but these errors were encountered: