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

OBPIH-4705 Fix Location chooser button #3456

Merged
merged 1 commit into from
Sep 12, 2022

Conversation

drodzewicz
Copy link
Collaborator

  • fix long location name text overlay
  • fix some padding/margin styling
  • added ability to close modal on click outside

- fix long location name text overlay
- fix some padding/margin styling
- added ability to close modal on click outside
@drodzewicz drodzewicz force-pushed the OBPIH-4705-location-functionality branch from 7241491 to 004cbd2 Compare September 9, 2022 10:27
@drodzewicz drodzewicz requested review from awalkowiak and kchelstowski and removed request for awalkowiak September 9, 2022 10:28
@@ -999,7 +999,7 @@ div.crossed-out .table-inner-row:before {
}

&::-webkit-scrollbar-thumb {
background-color: var(--blue-300);
background-color: var(--blue-301);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is the 301 a typo?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is not a typo, it is a different color
Unfortunately this color was not specified on the color pallet on figma so I named it myself.
Named it blue-301 because it is very similar to already defined color blue-300

@awalkowiak awalkowiak merged commit 0f38b6b into feature/ui-redesign Sep 12, 2022
@awalkowiak awalkowiak deleted the OBPIH-4705-location-functionality branch September 12, 2022 07:34
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