Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(tooltip): Eva style #1345

Merged
merged 16 commits into from
Apr 5, 2019
Merged

feat(tooltip): Eva style #1345

merged 16 commits into from
Apr 5, 2019

Conversation

yggg
Copy link
Contributor

@yggg yggg commented Apr 3, 2019

Please read and mark the following check list before creating a pull request:

Short description of what this resolves:

BREAKING CHANGES:

Following tooltip theme properties were renamed:
tooltip-bg -> tooltip-background-color
tooltip-fg -> tooltip-text-color
tooltip-font-size -> tooltip-text-font-size
tooltip-status-fg -> tooltip-[status]-text-color
tooltip-primary-bg -> tooltip-primary-background-color
tooltip-info-bg -> tooltip-info-background-color
tooltip-success-bg -> tooltip-success-background-color
tooltip-warning-bg -> tooltip-warning-background-color
tooltip-danger-bg -> tooltip-danger-background-color

Tooltip status class renamed from [status-name]-tooltip to
status-[status-name].

yggg added 11 commits April 3, 2019 10:16
BREAKING CHANGES:

Following tooltip theme properties were renamed:
tooltip-bg -> tooltip-background-color
tooltip-fg -> tooltip-text-color
tooltip-font-size -> tooltip-text-font-size
tooltip-status-fg -> tooltip-background-color, tooltip-text-color, tooltip-[status]-background-color, tooltip-[status]-text-color
BREAKING CHANGES:
Tooltip status class renamed from [status-name]-tooltip to
status-[status-name].
@yggg yggg requested a review from nnixaa April 3, 2019 07:34
@include nb-tooltip-status('warning');
@include nb-tooltip-status('info');
@each $status in nb-get-statuses() {
@include nb-tooltip-status($status);
Copy link
Collaborator

Choose a reason for hiding this comment

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

same question regarding empty status

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Answered here

nnixaa
nnixaa previously approved these changes Apr 4, 2019
@codecov
Copy link

codecov bot commented Apr 5, 2019

Codecov Report

Merging #1345 into next will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             next    #1345   +/-   ##
=======================================
  Coverage   81.63%   81.63%           
=======================================
  Files         237      237           
  Lines        7275     7275           
  Branches      626      626           
=======================================
  Hits         5939     5939           
  Misses       1150     1150           
  Partials      186      186
Impacted Files Coverage Δ
...work/theme/components/tooltip/tooltip.directive.ts 100% <100%> (ø) ⬆️
...work/theme/components/tooltip/tooltip.component.ts 100% <100%> (ø) ⬆️

@yggg yggg merged commit 5fba701 into akveo:next Apr 5, 2019
yggg added a commit that referenced this pull request Apr 17, 2019
BREAKING CHANGE:

Following tooltip theme properties were renamed:
tooltip-bg -> tooltip-background-color
tooltip-fg -> tooltip-text-color
tooltip-font-size -> tooltip-text-font-size
tooltip-status-fg -> tooltip-[status]-text-color
tooltip-primary-bg -> tooltip-primary-background-color
tooltip-info-bg -> tooltip-info-background-color
tooltip-success-bg -> tooltip-success-background-color
tooltip-warning-bg -> tooltip-warning-background-color
tooltip-danger-bg -> tooltip-danger-background-color

Tooltip status class renamed from [status-name]-tooltip to
status-[status-name].
yggg added a commit that referenced this pull request May 27, 2019
BREAKING CHANGE:

Following tooltip theme properties were renamed:
tooltip-bg -> tooltip-background-color
tooltip-fg -> tooltip-text-color
tooltip-font-size -> tooltip-text-font-size
tooltip-status-fg -> tooltip-[status]-text-color
tooltip-primary-bg -> tooltip-primary-background-color
tooltip-info-bg -> tooltip-info-background-color
tooltip-success-bg -> tooltip-success-background-color
tooltip-warning-bg -> tooltip-warning-background-color
tooltip-danger-bg -> tooltip-danger-background-color

Tooltip status class renamed from [status-name]-tooltip to
status-[status-name].
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.

None yet

2 participants