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

Add participant modal footer alignment fix #1949

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Aryakoste
Copy link

Summary

Ticket Link

Fixes mattermost/mattermost#27537
Jira https://mattermost.atlassian.net/browse/MM-58182

Before Screenshot:-
playbooksModalBeforeSS

After Screenshot:-
playbooksModalAfterSS

Checklist

- [ ] Telemetry updated
- [ ] Gated by experimental feature flag
- [ ] Unit tests updated

@mattermost-build
Copy link
Contributor

Hello @Aryakoste,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@mattermost-build mattermost-build requested review from a team and crspeller and removed request for a team October 19, 2024 14:35
@mattermost-build mattermost-build added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester Contributor labels Oct 19, 2024
@Aryakoste
Copy link
Author

This type of alignment issue is also occuring in other modals. Because the modal-footer is flex and the first element is :before content. I can make the change in generic modal if needed.

footer alignment

@Aryakoste
Copy link
Author

First the width 100% was not working i think it was not compiled properly now its working so changed to that.

@crspeller crspeller requested review from a team, devinbinnie and lindy65 and removed request for a team October 23, 2024 11:32
Copy link
Member

@devinbinnie devinbinnie left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @Aryakoste!

@devinbinnie devinbinnie removed the 2: Dev Review Requires review by a core committer label Oct 23, 2024
Copy link

@lindy65 lindy65 left a comment

Choose a reason for hiding this comment

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

Thanks @Aryakoste 🙌

cc @crspeller - this will need to be tested post merge once a new version of playbooks is cut

@lindy65 lindy65 added QA Deferred 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request Contributor Hacktoberfest QA Deferred
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alignment is off in "Add people_" modal of a Playbook run
5 participants