Skip to content
This repository was archived by the owner on Nov 3, 2021. It is now read-only.

update modal new skinny style #926

Merged
andrewhillcode merged 2 commits into from
Jan 14, 2021
Merged

update modal new skinny style #926

andrewhillcode merged 2 commits into from
Jan 14, 2021

Conversation

@wyvl
Copy link
Contributor

@wyvl wyvl commented Jan 13, 2021

INT-14641

The following MUST be checked prior to approval. If not relevant to your PR, remove the line from your description.

  • The styleguide.config.js has been updated to show examples/new functionality for the component
  • Test Coverage for any new or updated functionality
  • New and updated components have been tested locally in lapis and SPA and work as expected
  • This PR has been tagged with PATCH, MINOR, or MAJOR.
  • The component has data-component-type and data-component-context attributes following the standard
  • You have requested a cross-team review on this component so everyone knows it exists

@wyvl
Add skinny modal
db7cd1e 
INT-14641
@wyvl wyvl added the MAJOR Changes with backward incompatibility label Jan 13, 2021
@wyvl wyvl self-assigned this Jan 13, 2021
@wyvl wyvl changed the title update modal new style update modal new skinny style Jan 13, 2021
@wyvl wyvl requested a review from a team January 13, 2021 03:47
@wyvl wyvl added MINOR Extra functionality with backward compatible changes and removed MAJOR Changes with backward incompatibility labels Jan 13, 2021
@wyvl
Copy link
Contributor Author

wyvl commented Jan 13, 2021

Screen Shot 2021-01-13 at 2 11 00 pm

Screen Shot 2021-01-13 at 2 10 52 pm

Screen Shot 2021-01-13 at 2 10 00 pm

@wyvl wyvl force-pushed the technical/INT-14641-add-skinny-modal branch from 477b4a2 to 1b21bf1 Compare January 13, 2021 06:54
@wyvl
Copy link
Contributor Author

wyvl commented Jan 13, 2021 •
edited
Loading

Screen Shot 2021-01-13 at 4 55 04 pm

Screen Shot 2021-01-13 at 4 55 11 pm

andrewhillcode
andrewhillcode reviewed Jan 14, 2021
View reviewed changes
@andrewhillcode andrewhillcode merged commit 300a7c9 into intellihr:develop Jan 14, 2021
@intellibot-access intellibot-access bot mentioned this pull request Jan 14, 2021
Merged
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@andrewhillcode andrewhillcode andrewhillcode approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@wyvl wyvl

Labels

MINOR Extra functionality with backward compatible changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wyvl @andrewhillcode