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 workflow search from activity bar #17479

Closed
wants to merge 104 commits into from

Conversation

itisAliRH
Copy link
Member

@itisAliRH itisAliRH commented Feb 15, 2024

Requires: #16607. This PR removes the workflow search from the activity bar.
xref: #16607 (comment)

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@itisAliRH itisAliRH mentioned this pull request Feb 15, 2024
4 tasks
@mvdbeek
Copy link
Member

mvdbeek commented Feb 15, 2024

What I was saying in my comment is that the search bar conflicts with the search in the center panel. In no way do I want the workflow activity to be dropped.

@itisAliRH
Copy link
Member Author

What I was saying in my comment is that the search bar conflicts with the search in the center panel. In no way do I want the workflow activity to be dropped.

@mvdbeek Me neither, but the workflow activity without a search bar is used less and duplicates the masthead workflow.
However, #17289 I can implement this one and replace the search fields with favorite/recent workflows (using the ActivityPanel)

@mvdbeek
Copy link
Member

mvdbeek commented Feb 15, 2024

I think it is wrong that all activity bar items need a side panel. Also, the workflow activity did not have the side panel on 23.0 and that was perfectly fine. As commented, #17289 is a questionable idea that is unlikely to land in 24.0. Can you please, please, either fix the synchronization of the search or simply drop the search from the side panel.

@itisAliRH itisAliRH force-pushed the workflow-drop-side-search branch from 83a4ab5 to 6038e0d Compare February 15, 2024 16:30
@itisAliRH itisAliRH changed the title Drop workflow activity bar Remove workflow search from activity bar Feb 15, 2024
@itisAliRH itisAliRH force-pushed the workflow-drop-side-search branch from 6038e0d to 110bad5 Compare February 15, 2024 16:53
Copy link
Member

@mvdbeek mvdbeek left a comment

Choose a reason for hiding this comment

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

Thank you, that looks great!

@mvdbeek
Copy link
Member

mvdbeek commented Feb 16, 2024

https://github.com/galaxyproject/galaxy/actions/runs/7919456233/job/21623311924?pr=17479#step:11:2337 might be a relevant error ? I've triggered a rerun.

@itisAliRH
Copy link
Member Author

@mvdbeek It passed successfully, thanks. I've cherrypicked this PR changes to the main one and will close it now

@itisAliRH itisAliRH closed this Feb 16, 2024
@itisAliRH itisAliRH deleted the workflow-drop-side-search branch February 16, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants