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(datepicker): place active day above its siblings #3314

Merged

Conversation

peterblazejewicz
Copy link
Contributor

This is minor change that moves currently focus day within month view
above its siblings to let its outline be correctly rendered.
This is similar to minor stylistic corrections found in the BS:
https://git.io/fjyfh

The before rendering (Chrome/Mac OS):

Screenshot 2019-07-25 at 22 13 59

And after the change:
Screenshot 2019-07-25 at 22 14 23

Thanks!

This is minor change that moves currently focus day within month view
above its siblings to let its outline be correctly rendered.
This is similar to minor stylistic corrections found in the BS:
https://git.io/fjyfh

Thanks!
@codecov-io
Copy link

codecov-io commented Jul 25, 2019

Codecov Report

Merging #3314 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3314   +/-   ##
=======================================
  Coverage   90.88%   90.88%           
=======================================
  Files          95       95           
  Lines        2743     2743           
  Branches      510      510           
=======================================
  Hits         2493     2493           
  Misses        190      190           
  Partials       60       60
Flag Coverage Δ
#e2e 55.01% <ø> (ø) ⬆️
#unit 87.99% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b193bc6...d87f90a. Read the comment docs.

@benouat
Copy link
Member

benouat commented Sep 5, 2019

Nice catch! Thanks for this contribution 👍

Copy link
Member

@maxokorokov maxokorokov left a comment

Choose a reason for hiding this comment

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

LGTM!

@maxokorokov maxokorokov added this to the 5.1.2 milestone Sep 5, 2019
@benouat benouat merged commit eb26beb into ng-bootstrap:master Sep 5, 2019
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.

None yet

5 participants