Skip to content

Comments

Fix AA name overflows and improve tool tips for video options#158

Merged
wookieejedi merged 2 commits intoKnossosNET:mainfrom
wookieejedi:edit-AA-names
Feb 22, 2024
Merged

Fix AA name overflows and improve tool tips for video options#158
wookieejedi merged 2 commits intoKnossosNET:mainfrom
wookieejedi:edit-AA-names

Conversation

@wookieejedi
Copy link
Contributor

The new text PR makes two of the longer names in the Video options slightly cutoff. One option is to move the drop downs and check boxes more to the right, but then they would be out of alignment with the rest of the columns from other sections. Another, simpler option might be to just shorten the names, since they are rather long compared to other option names. From my tests, editing the names is the easiest and looks the best. Would be happy to refine as requested of course.

The new text PR makes two of the longer names in the Video options slightly cutoff. One option is to move the drop downs and check boxes more to the right, but then they would be out of alignment with the rest of the columns from other sections. Another, simpler option might be to just shorten the names, since they are rather long compared to other option  names. From my tests, editing the names is the easiest and looks the best.  Would be happy to refine as requested of course.
@wookieejedi wookieejedi added the cleanup Cleanup code or other aspect label Feb 19, 2024
Copy link
Contributor

@JohnAFernandez JohnAFernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@wookieejedi wookieejedi changed the title Fix AA name overflows Fix AA name overflows and improve tool tips for video options Feb 20, 2024
@wookieejedi wookieejedi merged commit a9af551 into KnossosNET:main Feb 22, 2024
@wookieejedi wookieejedi deleted the edit-AA-names branch February 22, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Cleanup code or other aspect

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants