mattermost / mattermost-developer-documentation

Mattermost developer documentation.
https://developers.mattermost.com
BSD 3-Clause "New" or "Revised" License
80 stars 389 forks source link

"Contribute" and "Integrate & Extend" Section Content Alignment Enhancement #1306

Closed iamRabia-N closed 10 months ago

iamRabia-N commented 10 months ago

Summary

This PR contains the solution for "Contribute" and "Integrate & Extend" Section Content Alignment Enhancement issue. It ensures a polished appearance by aligning the content to the center, enhancing overall visual appeal. This change aims to provide a more engaging and balanced layout, improving user experience and readability.

Ticket Link

Fixes https://github.com/mattermost/mattermost-developer-documentation/issues/1302

mattermost-build commented 10 months ago

Hello @iamRabia-N,

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.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement? Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

github-actions[bot] commented 10 months ago

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

cwarnermm commented 10 months ago

Thanks, @iamRabia-N! Are you open to signing the Mattermost CLA for your contribution?

emdecr commented 10 months ago

@cwarnermm Just checking with JuliAnn to see if she'd suggest to keep-as is because of card conventions we use across our web properties.

emdecr commented 10 months ago

@cwarnermm Looping back – confirmed with JuliAnn that the card design is consistent across web properties and the decision to left-align was for readability. Let me know if you want me to open the discussion up further and/or loop more folks in. Also appreciate the flag and suggestion @iamRabia-N !

cwarnermm commented 10 months ago

Thanks, @emdecr, for confirming creative intent for the Mattermost Developer Documentation site. Based on feedback from our Creative Team, I agree that a shared design component like this should remain as-is for consistency across Mattermost web-based assets.

@iamRabia-N - Though we won't be moving forward with this PR, your docs contribution and input about page design readability is appreciated. Thank you.