Skip to content

UI changes#265

Merged
Jasper-Harvey0 merged 6 commits into
mainfrom
ui-changes
Jul 22, 2026
Merged

UI changes#265
Jasper-Harvey0 merged 6 commits into
mainfrom
ui-changes

Conversation

@Jasper-Harvey0

Copy link
Copy Markdown
Collaborator
  • Updated font size on the user interaction buttons
  • Sequence end now shows "TEST PASS" or "TEST FAIL" in the image scene to make things really obvious.
image image

@Jasper-Harvey0

Copy link
Copy Markdown
Collaborator Author

We should have updated the version to 0.7.0 because of the python update....

@Jasper-Harvey0

Copy link
Copy Markdown
Collaborator Author

Oops, I have added the changes for adding user-id flag... @jcollins1983 I can either prune and force push, or we can do both changes in this one PR?

@jcollins1983

Copy link
Copy Markdown
Collaborator

Oops, I have added the changes for adding user-id flag... @jcollins1983 I can either prune and force push, or we can do both changes in this one PR?

It's not worth the churn to split it out. I'll review it under this one.

jcollins1983
jcollins1983 previously approved these changes Jul 22, 2026

@jcollins1983 jcollins1983 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks ok. Just one comment around the commenting in the new method in the UI if you care to take it on board.

Comment thread src/fixate/ui_gui_qt/ui_gui_qt.py Outdated
@Jasper-Harvey0

Copy link
Copy Markdown
Collaborator Author

Comments have been updated. Note that the version is going to 0.7.0 because we should have done that last time...

@jcollins1983 jcollins1983 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good.

@Jasper-Harvey0
Jasper-Harvey0 merged commit dbb5ce6 into main Jul 22, 2026
3 checks passed
@Jasper-Harvey0
Jasper-Harvey0 deleted the ui-changes branch July 22, 2026 05:58
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