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

Tweak spacing in responsive layout files for mobile #1359

Merged
merged 5 commits into from
May 29, 2024

Conversation

tnir
Copy link
Contributor

@tnir tnir commented Apr 29, 2024

Summary

Tweaks many spacing in responsive layout files mainly for mobile to reduce unexpected horizontal drifts on mobile screens.

This PR also helps developer experience by replacing home-made CSS selectors with Bootstrap 4's utilities.

Ticket Link

This change is a groundwork for Hugo layouts and not relevant to any specific feature(s).

This also helps developer experience by replacing
home-made CSS selectors with Bootstrap 4's utilities.

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
@mattermost-build
Copy link
Contributor

Hello @tnir,

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.

@tnir
Copy link
Contributor Author

tnir commented May 3, 2024

@cwarnermm Thoughts?

@cwarnermm cwarnermm added 2: Editor Review Requires review by an editor preview-environment Allow the preview environment to be generated for Pull Requests coming from fork repositories labels May 3, 2024
@cwarnermm cwarnermm self-requested a review May 3, 2024 12:44
Copy link

github-actions bot commented May 3, 2024

Newest code from cwarnermm has been published to preview environment for Git SHA b8dcfd6

1 similar comment
Copy link

github-actions bot commented May 3, 2024

Newest code from cwarnermm has been published to preview environment for Git SHA b8dcfd6

@cwarnermm
Copy link
Member

update-branch

@cwarnermm cwarnermm added preview-environment Allow the preview environment to be generated for Pull Requests coming from fork repositories and removed preview-environment Allow the preview environment to be generated for Pull Requests coming from fork repositories labels May 3, 2024
Copy link

github-actions bot commented May 3, 2024

Newest code from cwarnermm has been published to preview environment for Git SHA b8dcfd6

@tnir
Copy link
Contributor Author

tnir commented May 14, 2024

update-branch @cwarnermm

@cwarnermm cwarnermm requested a review from emdecr May 14, 2024 14:40
@cwarnermm
Copy link
Member

@emdecr - Are you open to taking a peek at this dev docs PR, please? I'd love to get your input on the proposed changes.

@emdecr
Copy link
Contributor

emdecr commented May 14, 2024

@cwarnermm Sure thing - will circle back within the next day or so.

Copy link
Contributor

@emdecr emdecr left a comment

Choose a reason for hiding this comment

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

@cwarnermm These standardizations make sense to me 👍
Looking around they don't seem to break anything.

Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thank you again, @tnir, for your patience on this docs PR.

@cwarnermm cwarnermm added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Editor Review Requires review by an editor preview-environment Allow the preview environment to be generated for Pull Requests coming from fork repositories labels May 29, 2024
@cwarnermm cwarnermm merged commit a07ab27 into mattermost:master May 29, 2024
5 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants