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

VL-16_Refactor-datepicker-range_Dmytro-Holdobin #70

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

Explicit12
Copy link
Collaborator

@Explicit12 Explicit12 requested a review from itsJohnnyGrid July 11, 2024 22:25
@Explicit12 Explicit12 self-assigned this Jul 11, 2024
@Explicit12 Explicit12 changed the title - Used UButton compVL-16_Refactor-datepicker-range_Dmytro-Holdobinonent in datepickerRanage VL-16_Refactor-datepicker-range_Dmytro-Holdobinonent in datepickerRanage Jul 11, 2024
@Explicit12 Explicit12 changed the title VL-16_Refactor-datepicker-range_Dmytro-Holdobinonent in datepickerRanage VL-16_Refactor-datepicker-range_Dmytro-Holdobin Jul 11, 2024
@Explicit12 Explicit12 force-pushed the VL-16_Refactor-datepicker-range_Dmytro-Holdobin branch from 9f700b3 to da595cf Compare July 18, 2024 15:28
flex h-[3.125rem] w-full cursor-pointer flex-col items-center justify-center rounded-lg bg-gray-100
px-1.5 py-2.5 text-center text-xs font-medium text-brand-900 hover:bg-brand-200
flex h-[3.125rem] w-full flex-col items-center justify-center bg-gray-100
px-1.5 py-2.5 text-xs font-medium hover:!bg-brand-200
Copy link
Contributor

Choose a reason for hiding this comment

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

  • set size="xs" and square after you can remove px-1.5 py-2.5 text-xs
  • remove font-medium
  • bg-gray-100 - lets remove it and keep brand default
  • keep default hover - remove this: hover:!bg-brand-200

So the final classes string should be: h-[3.125rem] w-full flex flex-col items-center justify-center

src/ui.form-date-picker-range/configs/default.config.js Outdated Show resolved Hide resolved
@Explicit12 Explicit12 force-pushed the VL-16_Refactor-datepicker-range_Dmytro-Holdobin branch from da595cf to 482df97 Compare July 19, 2024 08:35
@Explicit12 Explicit12 merged commit 0f07f6a into main Jul 19, 2024
1 check passed
@Explicit12 Explicit12 deleted the VL-16_Refactor-datepicker-range_Dmytro-Holdobin branch July 19, 2024 09:16
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.

2 participants