project-chip / zap

ZAP stands for ZCL Advanced Platform. It is used to configure clusters, attributes and other entities for Matter and ZigbeePro applications.
Apache License 2.0
133 stars 82 forks source link

Switching to endpoint configuration from the options, extensions and notifications page. #1382

Closed dhchandw closed 1 week ago

dhchandw commented 1 month ago
Screenshot 2024-07-31 at 9 17 15 AM

I'd appreciate any thoughts on this.

tbrkollar commented 1 month ago

notification page will not exist in the future. It will be a panel on the right site. 

paulr34 commented 1 month ago

I see a contradiction. The endpoint is still highlighted even though "Options" is Endpoint agnostic

What do you think about removing the highlight in this case. That way your feature will be more intuitive to the user because they are "reselecting" an endpoint and subsequently returning to the "Endpoint" view.

dhchandw commented 1 month ago

@paulr34 that makes sense. The endpoint shouldn't be highlighted unless we are on a page that is specific to it. Ideally, when we are on the options page for example, none of the endpoints should be highlighted but clicking on one of the endpoints should take us to the config page for that endpoint.

I think right now the endpoint that is highlighted just signifies that this is the endpoint config you go back to when you click the back button.