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

[2.1] Move hooks filter above table #8111

Open
wants to merge 2 commits into
base: release-2.1
Choose a base branch
from

Conversation

dragomano
Copy link
Contributor

Sometimes the hooks table layout is broken due to too long hook names. Moving the filter from the table header upwards solves this problem.

Before this change:
sshot-11

After:
sshot-12

@DiegoAndresCortes
Copy link
Member

DiegoAndresCortes commented Feb 20, 2024

Would #8059 work on this scenario too? (as a similar solution)

@dragomano
Copy link
Contributor Author

Yeah, if we update this rule:

#display_jump_to_select,
#message_index_jump_to_select,
#search_jump_to_select,
#quick_mod_jump_to_select,
#list_integration_hooks select {
	width: 29ch;
	overflow: hidden;
}

Can I use selector #list_integration_hooks select or should I add any ID for filter select field?

sshot-4

@DiegoAndresCortes
Copy link
Member

That selector should be good

Signed-off-by: Bugo <[email protected]>
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.

2 participants