Skip to content

app/vmui: replace log field copy action with exclude filter - #1663

Open
Loori-R wants to merge 1 commit into
masterfrom
vmui/log-field-exclude-filter
Open

app/vmui: replace log field copy action with exclude filter#1663
Loori-R wants to merge 1 commit into
masterfrom
vmui/log-field-exclude-filter

Conversation

@Loori-R

@Loori-R Loori-R commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Describe Your Changes

  • remove the Copy action
  • move Exclude from the overflow menu to the primary actions, replacing Copy
  • move field actions from the right side to the left, closer to the field name and the beginning of its value
  • truncate long group-by values to prevent them from overflowing the group header

Open Question

The actions are now closer to the field name and value, which makes them easier to access. However, the left side may look slightly more visually cluttered.

@hagen1778, @arturminchukov, what do you think about this placement?

Screenshots

Before image
After image

@Loori-R Loori-R added the web-ui Issues related to the user interface: layout, styling, usability, or UI-specific bugs. label Aug 3, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 6 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

@Loori-R
Loori-R force-pushed the vmui/log-field-exclude-filter branch from 4276189 to ae563aa Compare August 3, 2026 13:29

@arturminchukov arturminchukov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

web-ui Issues related to the user interface: layout, styling, usability, or UI-specific bugs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants