Skip to content

Conversation

@wangsizhu0504
Copy link
Member

What changes were proposed in this pull request

Issue Number: close #xxx

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

  1. App view table column add drag and drop
  2. The state fields in the table columns of the app view are of different lengths. We can align them according to the longest character in the list.
Snipaste_2024-01-17_15-17-13
  1. Display a description at the app name in the app table column when the mouse is hovered
  • final result *
image

Does this pull request potentially affect one of the following parts

  • Dependencies (does it add or upgrade a dependency): (yes / no)

Copy link
Member

@wolfboys wolfboys left a comment

Choose a reason for hiding this comment

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

LGTM

@wolfboys wolfboys merged commit e63562c into apache:dev-2.1.3 Jan 17, 2024
@wangsizhu0504 wangsizhu0504 deleted the improve/app branch February 17, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants