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

Refresh options UI #744

Merged
merged 3 commits into from
Jun 7, 2024
Merged

Refresh options UI #744

merged 3 commits into from
Jun 7, 2024

Conversation

tadast
Copy link
Collaborator

@tadast tadast commented Jun 4, 2024

  • reduce reliance on jQuery
  • make predefined options (when available) more prominent
  • use radio buttons for predefined options instead of a hover-based
    dropdown
  • make advanced options help modal prompt more prominent
  • Make the CTA sticky at the bottom so that it's always visible.
    Useful for instances with a lot of databases and predefined options,
    also smaller screens
    • enable customising options in SequenceServer Cloud as a plugin

Allow optional human readable descriptions to be added to predefined
settings. Maintain backwards compatibility with all old past formats.
This can be used in the front end by the Cloud version plugin to
build search params
- reduce reliance on jQuery
- enable customizing options in SequenceServer Cloud as a plugin
- make predefined options (when available) more prominent
- use radiobuttons for predefined options instead of a hover-based
  dropdown
- make advanced options help modal prompt more prominent
- Make the CTA sticky at the bottom so that it's always visible.
  Useful for instances with a lot of databases and predefined options,
  also smaller screens
@tadast tadast changed the title [WIP] Refresh options UI Refresh options UI Jun 7, 2024
@yannickwurm yannickwurm merged commit bbc50c7 into wurmlab:master Jun 7, 2024
3 of 4 checks passed
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