You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by "wu-sheng (via GitHub)" <gi...@apache.org> on 2023/03/22 09:12:26 UTC

[GitHub] [skywalking] wu-sheng commented on a diff in pull request #10576: Sync UI: redesign and implement new topology

wu-sheng commented on code in PR #10576:
URL: https://github.com/apache/skywalking/pull/10576#discussion_r1144448207


##########
docs/en/changes/changes.md:
##########
@@ -21,6 +21,7 @@
 #### UI
 * Revert: cpm5d function. This feature is cancelled from backend.
 * Fix: alerting link breaks on the topology.
+* Refactor Topology widget to make it more hierarchical.

Review Comment:
   ```suggestion
   * Refactor Topology widget to make it more hierarchical.
     1. Choose `User` as the first node.
     2. If `User` node is absent, choose the busiest node(which has the most calls of all).
     3. Do a left-to-right flow process.
     4. At the same level, list nodes from top to bottom in alphabetical order.
   ```
   
   @Fine0830 Please confirm whether this is the final resolution.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org