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

@Jasper-Harvey0Jasper-Harvey0 commented Jul 21, 2026

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.
imageimage

@Jasper-Harvey0

Jasper-Harvey0 commented Jul 21, 2026

Copy link
Copy Markdown
CollaboratorAuthor

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

@Jasper-Harvey0

Jasper-Harvey0 commented Jul 22, 2026

Copy link
Copy Markdown
CollaboratorAuthor

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

jcollins1983 commented Jul 22, 2026

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
View reviewed changes

@jcollins1983jcollins1983 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 threadsrc/fixate/ui_gui_qt/ui_gui_qt.py Outdated
@Jasper-Harvey0

Jasper-Harvey0 commented Jul 22, 2026

Copy link
Copy Markdown
CollaboratorAuthor

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

jcollins1983
jcollins1983 approved these changes Jul 22, 2026
View reviewed changes

@jcollins1983jcollins1983 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 mainJul 22, 2026
3 checks passed
@Jasper-Harvey0
Jasper-Harvey0 deleted the ui-changes branch July 22, 2026 05:58
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jcollins1983jcollins1983jcollins1983 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Jasper-Harvey0@jcollins1983