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

Improve NavigationBar and related tests #76

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

ahuston-0
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

@ahuston-0 ahuston-0 left a comment

Choose a reason for hiding this comment

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

Looks good to me! Just gotta see what Prof. Kruger thinks of the changes.

test/xp/testMultiTab.cc Outdated Show resolved Hide resolved
@ahuston-0 ahuston-0 added the enhancement New feature or request label May 26, 2022
@ahuston-0 ahuston-0 added this to the Summer 2022 Team milestone May 26, 2022
@ahuston-0 ahuston-0 requested a review from hydrodog May 27, 2022 16:00
TrentR2786 and others added 14 commits May 27, 2022 15:14
Fix test/CMakeLists.txt structure. The main branch has switched over to
using a tree of CMakeLists to clean the structure up a bit.

Also readded test2Tabs to the new system.

Signed-off-by: ahuston-0 <aliceghuston@gmail.com>
All currently compiling tests have been switched over to the new
grailmain.

Also, apparently test/xp/CMakeLists.txt was never added to the previous
commit.

Signed-off-by: ahuston-0 <aliceghuston@gmail.com>
Currently this throws an exception if a tab doesn't exist. My hope is
that the branch predictor will notice that the condition is true most of
the time and just return most of the time.

Signed-off-by: ahuston-0 <aliceghuston@gmail.com>
Copies over .gitattributes from main repo to ensure that files are
handled properly.

Fixes several tests dependent on GraphWidget and ButtonWidget to
ensure that the demos are kept in working condition (and the CI
is not spitting out errors every time a push is made)

Signed-off-by: ahuston-0 <aliceghuston@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants