Skip to content

Plans page improvements#122

Open
douglaswinter wants to merge 9 commits into
mainfrom
dw/plans_improvements
Open

Plans page improvements#122
douglaswinter wants to merge 9 commits into
mainfrom
dw/plans_improvements

Conversation

@douglaswinter

Copy link
Copy Markdown
Collaborator

Fixes some issues in #97, namely alphabetising list of plans and making that list independently scrollable. Other layout improvements as well, and since @atlas/blueapi-ui depends on @atlas/blueapi-query it now retrieves plans internally.

@douglaswinter douglaswinter changed the title Dw/plans improvements Plans page improvements Jul 15, 2026
@douglaswinter

Copy link
Copy Markdown
Collaborator Author

Things in #97 not addressed:

  • 'favourites' feature. Potentially complex and I don't believe it's good use of our time as 1) favourite complex plans will be backing standalone features and 2) favourite trivial plans (equivalent to 'pos this there') will be used primarily via console when available.
  • plans not rendering well. This could be worked on, but the feature was meant to be auto-generated uis based on blueapi schema, and so this should be addressed by improving that schema, if it's important.

@douglaswinter
douglaswinter requested a review from EmsArnold July 15, 2026 12:23

@EmsArnold EmsArnold left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

New layout looks good, and looks to function well on different sized screens! Tests need to be addressed, some ideas in comments.

Comment thread packages/blueapi-ui/src/PlanBrowser/PlanBrowser.test.tsx Outdated
Comment thread packages/blueapi-ui/src/PlanBrowser/PlanParameters.test.tsx
@douglaswinter
douglaswinter force-pushed the dw/plans_improvements branch from 68b588c to 49aed63 Compare July 17, 2026 15:26
@douglaswinter
douglaswinter requested a review from EmsArnold July 17, 2026 15:27
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