[ 
https://issues.apache.org/jira/browse/ARTEMIS-3653?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Erwin Dondorp updated ARTEMIS-3653:
-----------------------------------
    Summary: BrokerDiagram improvements, a.o. highlight the (real) current 
broker  (was: BrokerDiagram should highlight the (real) current broker)

> BrokerDiagram improvements, a.o. highlight the (real) current broker
> --------------------------------------------------------------------
>
>                 Key: ARTEMIS-3653
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3653
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>          Components: Web Console
>    Affects Versions: 2.20.0
>            Reporter: Erwin Dondorp
>            Priority: Minor
>
> When viewing the broker diagram with up to one or two master nodes (and 
> optionally a slave node for each) then it is unclear which node is the 
> current node.
> When there are 3 or more master nodes, the connections between the nodes 
> provide a clue (as the one with multiple connections is from the current 
> pair). but still it is unclear whether the masternode or the slavenode of the 
> current pair is the current node.
> The code shows that an attempt was made to highlight the current node. 
> actually always the master node from the current pair is highlighted. since 
> the styling is the same, that is not noticeable.
> The (upcoming) PR for this change will introduce proper highlighting of the 
> current node. Simultaneously, the previous implementation is fixed where 
> needed.
> Details:
> * (unrelated) do not show ":61616" in the broker name
> * (unrelated) do not show quotes around addresses and queues
> * The node-type "ThisBroker" is split into "ThisMasterBroker" and 
> "ThisSlaveBroker"
> * "ThisMasterBroker" has the same style as "MasterBroker", but just with a 
> black border
> * "ThisSlaveBroker" has the same style as "SlaveBroker", but just with a 
> black border
> * TODO: determine whether the broker itself is master or slave
> * TODO: apply the "new" styles only to the current broker



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to