Skip to content

ui: refactor labels with tooltip in forms#5133

Merged
yadvr merged 18 commits into
apache:mainfrom
shapeblue:ui-refactor-formitemtooltiplabel
Jul 15, 2021
Merged

ui: refactor labels with tooltip in forms#5133
yadvr merged 18 commits into
apache:mainfrom
shapeblue:ui-refactor-formitemtooltiplabel

Conversation

@shwstppr
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr commented Jun 18, 2021

Description

Adds new Vue component TooltipLabel for easier re-use
Moved TooltipButton.vue (and TooltipLabel.vue) to components/widgets folder.
Additionally, cosmetic changes to add compute/service offering, network offering and shared network form.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

Screenshot from 2021-07-01 13-23-54
Screenshot from 2021-07-01 13-32-42

How Has This Been Tested?

Adds new Vue component TooltipLabel for easier re-use

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr added this to the 4.16.0.0 milestone Jun 18, 2021
shwstppr and others added 12 commits June 21, 2021 12:56
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.kumar@shapeblue.com>
Signed-off-by: Abhishek Kumar <abhishek.kumar@shapeblue.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr marked this pull request as ready for review July 1, 2021 10:28
@shwstppr
Copy link
Copy Markdown
Contributor Author

shwstppr commented Jul 1, 2021

@blueorangutan ui

@shwstppr shwstppr requested review from Pearl1594 and utchoang and removed request for Pearl1594 July 1, 2021 10:28
Copy link
Copy Markdown

@utchoang utchoang left a comment

Choose a reason for hiding this comment

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

LGTM!

@davidjumani
Copy link
Copy Markdown
Contributor

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@davidjumani a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

UI build: ✖️
(SL-JID-353)

shwstppr added 2 commits July 14, 2021 09:19
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Comment thread ui/src/views/compute/CreateKubernetesCluster.vue Outdated
Comment thread ui/src/views/storage/CreateVolume.vue Outdated
Comment thread ui/src/views/storage/UploadLocalVolume.vue Outdated
Comment thread ui/src/views/storage/UploadLocalVolume.vue Outdated
shwstppr and others added 3 commits July 14, 2021 09:58
Co-authored-by: Pearl Dsilva <pearl1594@gmail.com>
Co-authored-by: davidjumani <dj.davidjumani1994@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@apache apache deleted a comment from blueorangutan Jul 14, 2021
@apache apache deleted a comment from blueorangutan Jul 14, 2021
@shwstppr
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5133 (SL-JID-355)

Copy link
Copy Markdown
Contributor

@Pearl1594 Pearl1594 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown

@utchoang utchoang left a comment

Choose a reason for hiding this comment

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

LGTM!

@yadvr yadvr merged commit 041948c into apache:main Jul 15, 2021
@shwstppr shwstppr mentioned this pull request Jul 19, 2021
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants