Skip to content

Improvement of Orders.Web#1093

Merged
dbeuchler merged 7 commits intodevfrom
fix/operation-count-and-progress
Feb 27, 2026
Merged

Improvement of Orders.Web#1093
dbeuchler merged 7 commits intodevfrom
fix/operation-count-and-progress

Conversation

@dbeuchler
Copy link
Member

@dbeuchler dbeuchler commented Feb 23, 2026

close #1080 #1079 #1095 #1081

Screenshot 2026-02-26 at 15 58 18 Screenshot 2026-02-26 at 15 58 32 Screenshot 2026-02-26 at 15 58 52 Screenshot 2026-02-26 at 15 59 52

@dbeuchler dbeuchler added this to the Framework 10.x milestone Feb 23, 2026
@dbeuchler dbeuchler self-assigned this Feb 23, 2026
@dbeuchler dbeuchler added the bug Something isn't working label Feb 23, 2026
@dbeuchler dbeuchler force-pushed the fix/operation-count-and-progress branch from e2d8abc to e982531 Compare February 23, 2026 13:38
@dbeuchler dbeuchler force-pushed the fix/operation-count-and-progress branch from e982531 to b84fad2 Compare February 23, 2026 18:32
@dbeuchler dbeuchler changed the title Fixed operation state indicator and restored colors Fixed operation state, restored colors, hide-completed Feb 24, 2026
@1nf0rmagician 1nf0rmagician changed the base branch from dev to future February 25, 2026 06:58
@1nf0rmagician 1nf0rmagician changed the base branch from future to dev February 25, 2026 06:58
Copy link
Member

@1nf0rmagician 1nf0rmagician left a comment

Choose a reason for hiding this comment

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

Could we split this PR in the fixes and the addtion of the hide/diplay completed feature? The fixes seam fine to me, even though the html part is really hard to review the way github shows the diff unfortunately 🙈

The design of hide-completed feature directly violates the material design guideline, however. As only one FAB button should be displayed per screen, source. Also adding the feature should already take into account that there is another pending feature request to filter orders e.g. by product, recipe, operation number which serve a similar purpose and hence belong into a coherent "how do we display filter options" concept for the UI.
@MathoMathiasCamara would be interested in your thoughts on that matter. Maybe a top-bar (like shifts UI), maybe a bottom-bar? The products UI has filters in the left component, which is unfortunately not applicable here. The Visual Instructions UI basically filters instructions by client, which is configured through a settings button in the top right...

Also, considering that there are more filter options coming, I think separating the local storage interaction into a service would be nice

@MathoMathiasCamara
Copy link
Contributor

Could we split this PR in the fixes and the addtion of the hide/diplay completed feature? The fixes seam fine to me, even though the html part is really hard to review the way github shows the diff unfortunately 🙈

The design of hide-completed feature directly violates the material design guideline, however. As only one FAB button should be displayed per screen, source. Also adding the feature should already take into account that there is another pending feature request to filter orders e.g. by product, recipe, operation number which serve a similar purpose and hence belong into a coherent "how do we display filter options" concept for the UI. @MathoMathiasCamara would be interested in your thoughts on that matter. Maybe a top-bar (like shifts UI), maybe a bottom-bar? The products UI has filters in the left component, which is unfortunately not applicable here. The Visual Instructions UI basically filters instructions by client, which is configured through a settings button in the top right...

Also, considering that there are more filter options coming, I think separating the local storage interaction into a service would be nice

Yes i think having the top bar here is a good idea.

@dbeuchler
Copy link
Member Author

discussed with @MathoMathiasCamara we will add a top bar like in jobs or workplans.

@dbeuchler dbeuchler marked this pull request as draft February 25, 2026 10:10
@dbeuchler
Copy link
Member Author

@MathoMathiasCamara proposal
Screenshot 2026-02-25 at 20 43 21

@dbeuchler
Copy link
Member Author

next iteration after discussion with @MathoMathiasCamara

Screenshot 2026-02-26 at 14 09 05

@MathoMathiasCamara
Copy link
Contributor

next iteration after discussion with @MathoMathiasCamara
Screenshot 2026-02-26 at 14 09 05

For the main navigation inside a module this is good.

@dbeuchler dbeuchler changed the title Fixed operation state, restored colors, hide-completed First improvement of Orders.Web Feb 26, 2026
@dbeuchler dbeuchler changed the title First improvement of Orders.Web Improvement of Orders.Web Feb 26, 2026
@dbeuchler dbeuchler force-pushed the fix/operation-count-and-progress branch from 348f561 to 719c697 Compare February 26, 2026 14:51
@dbeuchler dbeuchler marked this pull request as ready for review February 26, 2026 15:00
@dbeuchler
Copy link
Member Author

dbeuchler commented Feb 26, 2026

@MathoMathiasCamara I have clean everything and extracted the progress-bar into a component for reuse. The only thing which I don't like is that the side-bar-nav style is currently "self-written". The density settings makes it hard to use existing components like buttons. The style of the side-bar was mainly designed like here: https://m3.material.io

After our UX discussion we have better ideas where we want to go.

Could you review the code for me?

@MathoMathiasCamara
Copy link
Contributor

@MathoMathiasCamara I have clean everything and extracted the progress-bar into a component for reuse. The only thing which I don't like is that the side-bar-nav style is currently "self-written". The density settings makes it hard to use existing components like buttons. The style of the side-bar was mainly designed like here: https://m3.material.io

After our UX discussion we have better ideas where we want to go.

Could you review the code for me?

This looks nice. we could prepare this as a template for the other UIs.

@dbeuchler dbeuchler merged commit c254953 into dev Feb 27, 2026
14 of 15 checks passed
@dbeuchler dbeuchler deleted the fix/operation-count-and-progress branch February 27, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Operation state indicator is always black

3 participants