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

Fix/521 override dialog styles #532

Merged
merged 5 commits into from
Dec 18, 2023
Merged

Conversation

ektaghag-eaton
Copy link
Contributor

Fixes 521 .

Changes proposed in this Pull Request:

  • Add sx prop to dialogConfig
  • Update changelog

Screenshots / Screen Recording (if applicable)

Screenshot 2023-12-13 at 6 15 48 PM

To Test:

  • Throw an error from login auth UI actions
  • Add the below code in login-workflow/example/src/screens/Login.tsx

<LoginScreen projectImage={<img src={EatonLogo} alt="logo" style={{ maxHeight: 80 }} />} header={<DebugComponent />} errorDisplayConfig={{ mode: 'dialog', messageBoxConfig: { dismissible: true, position: 'top', sx: { backgroundColor: 'green' }, }, dialogConfig: { title: 'Change Title Color', sx: { '& .MuiTypography-h6': { color: 'red', }, }, }, }} />

Any specific feedback you are looking for?

  • Sorry for unformatted code.

@github-actions github-actions bot added the brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering label Dec 13, 2023
Copy link

github-actions bot commented Dec 13, 2023

Visit the preview URL for this PR (updated for commit dc01f3d):

https://blui-react-login--pr532-fix-521-override-dia-p6bpsekg.web.app

(expires Wed, 20 Dec 2023 06:59:02 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 1e9064d2e35ed37fe01e053587ea5f209719a043

Copy link
Collaborator

@JeffGreiner-eaton JeffGreiner-eaton left a comment

Choose a reason for hiding this comment

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

look good. not sure if this is a fixed in change log since it wasn't a requirement from the initial implementation.

@ektaghag-eaton
Copy link
Contributor Author

ektaghag-eaton commented Dec 14, 2023

look good. not sure if this is a fixed change log since it wasn't a requirement from the initial implementation.

Sure jeff will remove it. @surajeaton , any suggestions?

login-workflow/src/components/Error/ErrorManager.tsx Outdated Show resolved Hide resolved
login-workflow/src/components/Error/ErrorManager.tsx Outdated Show resolved Hide resolved
login-workflow/CHANGELOG.md Show resolved Hide resolved
login-workflow/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@surajeaton surajeaton left a comment

Choose a reason for hiding this comment

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

Looks good.

@surajeaton surajeaton merged commit 9f2aa8e into dev Dec 18, 2023
7 checks passed
@surajeaton surajeaton deleted the fix/521-override-dialog-styles branch December 18, 2023 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering
Development

Successfully merging this pull request may close these issues.

Unable to override the styles in error dialog
3 participants