day8 / re-com

A ClojureScript library of reusable components for Reagent
https://re-com.day8.com.au
MIT License
796 stars 147 forks source link

Datepicker header height increases when a disabled next or previous year button is displayed #313

Closed Gregg8 closed 2 years ago

Gregg8 commented 2 years ago

Correct header height image

Increased header height when disabled Next Year button is shown (same with Previous Year) image

You cannot demonstrate this in the demo page because it does not allow you to set the :minimum or :maximum args.

MawiraIke commented 2 years ago

This issue seems to be caused by padding from re-com.css applied to rc-datepicker-disabled and rc-datepicker-unselectable classes. The padding would also be applied to disabled dates but is overwritten by padding from bootstrap.css as shown in the diagram below. Simply, the padding is only used by the following classes in re-com; rc-datepicker-prev-year, rc-datepicker-prev-month, rc-datepicker-next-month and rc-datepicker-next-year which are the icons that cause this issue. I would suggest removing the padding from re-com.css.

preview

Pushing a fix for this.