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

Fixes to executable table layout #2145

Closed
wants to merge 8 commits into from

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Sep 17, 2023

Still WIP but some questions on layout

image
Do we think the different tables are improvement or cluther?

Should I re-sort the tables on enabled/disabled. I think given that we have the search option there it might make it cleaner.

The code tag looks like an improvement:
image

If we want this we should still remove the type from the table itself.
This needs to be further extended as debug and default compare scripts
are now shown as unused.
@meisterT
Copy link
Member

I would try to have two tables, one with used and one unused executables and separating them by icon (and sorting) instead and see what looks better.

@vmcj
Copy link
Member Author

vmcj commented Sep 17, 2023

image

I've now also labeled executables as disabled when the language they belong to are not allowed to be submitted.

@vmcj vmcj requested a review from meisterT September 17, 2023 20:38
@vmcj vmcj closed this Oct 5, 2023
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