Skip to content

Make QueueTaskModal more compact (collapsible resource override)#1577

Open
deep1401 wants to merge 7 commits intomainfrom
fix/queuetaskmodal
Open

Make QueueTaskModal more compact (collapsible resource override)#1577
deep1401 wants to merge 7 commits intomainfrom
fix/queuetaskmodal

Conversation

@deep1401
Copy link
Member

No description provided.

@paragon-review
Copy link

Paragon Review Skipped

Hi @deep1401! Your Polarity credit balance is insufficient to complete this review.

Please visit https://home.polarity.cc to add more credits and continue using Paragon reviews.

Copy link
Member

@dadmobile dadmobile left a comment

Choose a reason for hiding this comment

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

Looks like we moved resources to the bottom? And then when I click on the arrow the window doesn't scroll down so it's not clear that I expanded anything.

Some brainstorm thoughts:

  1. I would either move this back to where it was and just have it expand/collapse, or if we leave it at the bottom then maybe the window should scroll down automatically when you expand?
  2. Most of my queue window is the task parameters (for most tasks this is bigger than the resource overrides). I wonder if it's insane to collapse those as well?

Copy link
Member Author

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

Looks like we moved resources to the bottom? And then when I click on the arrow the window doesn't scroll down so it's not clear that I expanded anything.

Some brainstorm thoughts:

  1. I would either move this back to where it was and just have it expand/collapse, or if we leave it at the bottom then maybe the window should scroll down automatically when you expand?
  2. Most of my queue window is the task parameters (for most tasks this is bigger than the resource overrides). I wonder if it's insane to collapse those as well?

Did both (scroll into view for 1)

@deep1401 deep1401 requested a review from dadmobile March 19, 2026 20:39
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