Skip to content

[Feature][UI Next][V1.0.0-Alpha] The label of the form in the unified project is arranged up and down.#8991

Merged
songjianet merged 2 commits into
apache:devfrom
labbomb:labbomb
Mar 18, 2022
Merged

[Feature][UI Next][V1.0.0-Alpha] The label of the form in the unified project is arranged up and down.#8991
songjianet merged 2 commits into
apache:devfrom
labbomb:labbomb

Conversation

@labbomb
Copy link
Copy Markdown
Member

@labbomb labbomb commented Mar 18, 2022

Purpose of the pull request

Please refer to the main issue #8990.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

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

(or)

This change added tests and can be verified as follows:

@songjianet songjianet added feature new feature UI ui and front end related labels Mar 18, 2022
ref='detailFormRef'
require-mark-placement='left'
label-placement='left'
label-width={180}
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.

label-width is not required in top-down forms.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ok,i will fix it

@songjianet songjianet changed the title [Feature][UI Next]The label of the form in the unified project is arranged up and down [Feature][UI Next][V1.0.0-Alpha] The label of the form in the unified project is arranged up and down. Mar 18, 2022
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Copy Markdown
Member

@songjianet songjianet left a comment

Choose a reason for hiding this comment

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

LGTM

@songjianet songjianet merged commit 7d94ada into apache:dev Mar 18, 2022
@labbomb labbomb deleted the labbomb branch March 18, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature new feature UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants