Skip to content

[Feature] Update package/lynx-trace submodule with bug fix and UI adjustment#71

Merged
KananSu merged 1 commit into
lynx-family:mainfrom
KananSu:p/kanan/trace_plugin_upgrade_submodule
Jul 15, 2025
Merged

[Feature] Update package/lynx-trace submodule with bug fix and UI adjustment#71
KananSu merged 1 commit into
lynx-family:mainfrom
KananSu:p/kanan/trace_plugin_upgrade_submodule

Conversation

@KananSu

@KananSu KananSu commented Jul 14, 2025

Copy link
Copy Markdown
Collaborator

Summary:
This update to the lynx-trace submodule includes the following changes:

  1. Fixed a bug in renderDoFrameTag where the condition incorrectly checked if filteredTraceSet.has(value.id) was not undefined, now corrected to use the boolean result directly.
  2. Adjusted the argument order in ThreadSliceDetailsPanel's render method, moving args and description before precFlows and followingFlows to improve consistency and clarify layout structure. This change does not affect functionality.

…ustment

Summary:
This update to the lynx-trace submodule includes the following changes:
1. Fixed a bug in renderDoFrameTag where the condition incorrectly checked
if filteredTraceSet.has(value.id) was not undefined, now corrected to use the
boolean result directly.
2. Adjusted the argument order in ThreadSliceDetailsPanel's render method, moving
args and description before precFlows and followingFlows to improve consistency
and clarify layout structure. This change does not affect functionality.
@KananSu

KananSu commented Jul 14, 2025

Copy link
Copy Markdown
Collaborator Author

@talisk Hi, could you please take a look at this PR? Thanks!

@KananSu KananSu enabled auto-merge July 14, 2025 11:39
@KananSu KananSu added this pull request to the merge queue Jul 15, 2025
Merged via the queue into lynx-family:main with commit 7caa13a Jul 15, 2025
2 checks passed
@KananSu KananSu deleted the p/kanan/trace_plugin_upgrade_submodule branch July 15, 2025 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants