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

Remove unnecessary text tms coil #842

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Hafsa-shoaib989
Copy link
Contributor

This PR fixes issue #836, which removes the unnecessary "Edit Configuration" text from the TMS Coil tab in coregistration and repositions the "Preferences" button to the left where the removed text was.

Changes Made:

  • Updated the task_navigator.py file.

Testing:

  • Verified that the "Preferences" button is correctly positioned on the TMS Coil tab.
  • Ensured there are no alignment issues or UI breakages after the change.

Hi @henrikkauppi @vhosouza @tfmoraes please review this new Pull request.

@vhosouza vhosouza requested a review from rmatsuda September 12, 2024 07:07
@vhosouza vhosouza added the ui/ux Bug or enhancement related to user interface and user experience label Sep 12, 2024
Copy link
Collaborator

@henrikkauppi henrikkauppi left a comment

Choose a reason for hiding this comment

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

I tested it and the buttons and other UI components work fine. This can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui/ux Bug or enhancement related to user interface and user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants