-
Notifications
You must be signed in to change notification settings - Fork 26
Sequence Diagram tooltips #2326
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
b148921
include audit section displaying audit messages
PhilBastian 105cc68
show status icons
PhilBastian 337fdef
Merge branch 'master' into si_integration
PhilBastian dd2cd69
add feature flag
PhilBastian 36d7815
format message type as per SI
PhilBastian b281cf3
change name from audit to all messages and include message view route
PhilBastian f706a7f
display time_sent and processing_time values
PhilBastian db87e15
fix the body of the popup message in retry dialog
soujay ef84c78
Revert "fix the body of the popup message in retry dialog"
soujay 2616724
enable auto refresh to be paused and to be configured after creation
PhilBastian 0065abb
manual/auto refresh control
PhilBastian 6933c0f
Merge branch 'si_integration' of https://github.com/Particular/Servic…
PhilBastian 10ae9af
stripe results grid
PhilBastian 0a510d0
change to use emits
PhilBastian ba62324
remove commented code
PhilBastian 60e4bfa
add paging
PhilBastian 322f801
implement server side sorting
PhilBastian 9cf0ac4
scroll on list instead of view, allowing the table headers to always …
PhilBastian 338d370
Merge branch 'master' into si_integration
PhilBastian 3b206eb
change to use messages routing
PhilBastian 9e77b04
increase default page size
PhilBastian be36fd2
add failedmessage redirect and remove unused message components
PhilBastian c2f286e
start converting conversation model from SI
PhilBastian 83592dc
order messages
PhilBastian 0a07896
Merge branch 'master' into sequence_diagram
PhilBastian 6a3fdab
add endpoint host versions
PhilBastian b2e1204
convert handleritem logic from SI
PhilBastian 72df26f
convert routedMessages (arrows) from SI
PhilBastian a89730a
display endpoints
PhilBastian 90a0052
timeline
PhilBastian eb5ffa5
handlers
PhilBastian 7350293
arrow (left to right)
PhilBastian 19171da
expand SVG to fit larger diagrams
PhilBastian f24f540
fix handler map
PhilBastian a9b6488
fix messagetype text side
PhilBastian 5108f2a
replace props and emits with a store
PhilBastian 39929c2
highlight on handlers
PhilBastian 29f435a
implement route highlight
PhilBastian 2de64d6
hide sequence diagram behind feature flag
PhilBastian c926f54
Merge branch 'master' into sequence_diagram
PhilBastian 9303f63
support local and response message flow
PhilBastian ff1666f
handle long message names being pushed off the svg surface
PhilBastian 35e4232
handle handler messagetext offset less than 0 (buggy)
PhilBastian b32a4fd
display event as dashed line
PhilBastian 9645d4a
handle events in SequenceDiagram
PhilBastian ed3aa5a
endpoint tooltips
PhilBastian 0b0f590
address comments from review
PhilBastian c62f833
Merge branch 'sequence_diagram' into reverse_local_events_timeout
PhilBastian 821947e
add comment for clarity
PhilBastian 799eebb
Merge branch 'master' into reverse_local_events_timeout
PhilBastian 20f3a71
fix merge issue
PhilBastian ec1a5b7
simplify name uniqueness
PhilBastian 204c6c6
Merge branch 'reverse_local_events_timeout' into sd_tooltips
PhilBastian 948a63e
Merge branch 'master' into sd_tooltips
PhilBastian b77f819
alignment to match SI
PhilBastian 632a5cd
implement handler tooltip and move tooltip creation outside of render…
PhilBastian f969bea
implement handler tooltip and move tooltip creation outside of render…
PhilBastian 85130da
Merge branch 'sd_tooltips' of https://github.com/Particular/ServicePu…
PhilBastian 23b3a28
route tooltip
PhilBastian 499fa1f
fix timeout detection
PhilBastian 881c51a
fix sequence diagram not re-rendering on tab switch away and back
PhilBastian 75b0fea
fix startX offset calculation
PhilBastian a7fd209
Merge branch 'master' into sd_tooltips
PhilBastian 184a567
incorporate sequence diagram into new message view
PhilBastian File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
35 changes: 35 additions & 0 deletions
35
src/Frontend/src/components/messages2/SequenceDiagram/EndpointTooltip.vue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,35 @@ | ||
<script setup lang="ts"> | ||
import { Endpoint } from "@/resources/SequenceDiagram/Endpoint"; | ||
|
||
defineProps<{ endpoint: Endpoint }>(); | ||
</script> | ||
|
||
<template> | ||
<div class="endpoint-tooltip"> | ||
<label>Name:</label> | ||
<span>{{ endpoint.name }}</span> | ||
<label>NSB Version:</label> | ||
<span>{{ endpoint.version }}</span> | ||
<label>Host:</label> | ||
<span>{{ endpoint.host }}</span> | ||
</div> | ||
</template> | ||
|
||
<style> | ||
.endpoint-tooltip { | ||
display: grid; | ||
grid-template-columns: auto auto; | ||
column-gap: 0.5em; | ||
} | ||
|
||
.endpoint-tooltip label { | ||
grid-column: 1; | ||
justify-self: end; | ||
font-weight: bold; | ||
color: #b3b3b3; | ||
} | ||
|
||
.endpoint-tooltip span { | ||
word-break: break-all; | ||
} | ||
</style> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
54 changes: 54 additions & 0 deletions
54
src/Frontend/src/components/messages2/SequenceDiagram/HandlerTooltip.vue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,54 @@ | ||
<script setup lang="ts"> | ||
import { useFormatTime } from "@/composables/formatter"; | ||
import { Handler } from "@/resources/SequenceDiagram/Handler"; | ||
|
||
defineProps<{ handler: Handler }>(); | ||
|
||
function formatTime(milliseconds: number) { | ||
const time = useFormatTime(milliseconds); | ||
return `${time.value} ${time.unit}`; | ||
} | ||
</script> | ||
|
||
<template> | ||
<div v-if="handler.id === 'First'">Start of Conversation</div> | ||
<div v-else class="handler-tooltip"> | ||
<div class="title">Processing of Message</div> | ||
<div class="details"> | ||
<label>Processing Time:</label> | ||
<span>{{ formatTime(handler.processingTime ?? 0) }}</span> | ||
<label>Processing Of:</label> | ||
<span>{{ handler.name }}</span> | ||
<label v-if="handler.partOfSaga">Sagas Invoked:</label> | ||
<span v-if="handler.partOfSaga">{{ handler.partOfSaga }}</span> | ||
</div> | ||
</div> | ||
</template> | ||
|
||
<style> | ||
.handler-tooltip { | ||
display: flex; | ||
flex-direction: column; | ||
} | ||
|
||
.handler-tooltip .title { | ||
font-weight: bold; | ||
} | ||
|
||
.handler-tooltip .details { | ||
display: grid; | ||
grid-template-columns: auto auto; | ||
column-gap: 0.5em; | ||
} | ||
|
||
.handler-tooltip label { | ||
grid-column: 1; | ||
justify-self: end; | ||
font-weight: bold; | ||
color: #b3b3b3; | ||
} | ||
|
||
.handler-tooltip span { | ||
word-break: break-all; | ||
} | ||
</style> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
49 changes: 49 additions & 0 deletions
49
src/Frontend/src/components/messages2/SequenceDiagram/RouteTooltip.vue
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,49 @@ | ||
<script setup lang="ts"> | ||
import { RoutedMessage, RoutedMessageType } from "@/resources/SequenceDiagram/RoutedMessage"; | ||
|
||
defineProps<{ routedMessage: RoutedMessage }>(); | ||
</script> | ||
|
||
<template> | ||
<div class="route-tooltip"> | ||
<div class="title">{{ RoutedMessageType[routedMessage.type] }}</div> | ||
<div class="details"> | ||
<label>Message Type:</label> | ||
<span>{{ routedMessage.name }}</span> | ||
<label>Message Id:</label> | ||
<span>{{ routedMessage.messageId }}</span> | ||
<label>Sent From:</label> | ||
<span>{{ `${routedMessage.sending.name}@${routedMessage.sending.host}` }}</span> | ||
<label>Sent To:</label> | ||
<span>{{ `${routedMessage.receiving.name}@${routedMessage.receiving.host}` }}</span> | ||
</div> | ||
</div> | ||
</template> | ||
|
||
<style> | ||
.route-tooltip { | ||
display: flex; | ||
flex-direction: column; | ||
} | ||
|
||
.route-tooltip .title { | ||
font-weight: bold; | ||
} | ||
|
||
.route-tooltip .details { | ||
display: grid; | ||
grid-template-columns: auto auto; | ||
column-gap: 0.5em; | ||
} | ||
|
||
.route-tooltip label { | ||
grid-column: 1; | ||
justify-self: end; | ||
font-weight: bold; | ||
color: #b3b3b3; | ||
} | ||
|
||
.route-tooltip span { | ||
word-break: break-all; | ||
} | ||
</style> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this be a component that is reused?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we could use it in the list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and probably other places
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
created an idea card