You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@seata.apache.org by "navjotmaan (via GitHub)" <gi...@apache.org> on 2025/12/23 13:37:38 UTC

[PR] docs: update homepage content font-size to 18px and improve visibility [incubator-seata-website]

navjotmaan opened a new pull request, #1057:
URL: https://github.com/apache/incubator-seata-website/pull/1057

   
   ## What is the purpose of the change
   
   The purpose of this change is to unify the typography and font sizes across the home page to improve readability and user experience. Specifically, it addresses inconsistencies where some content was too small (14px) or had low contrast, as reported in issue #875.
   
   ## Brief changelog
   
   - Updated home page content (paragraphs and feature lists) font-size to 18px.
   
   - Changed content typography to sans-serif.
   
   - Improved text visibility by darkening the font color.
   
   - Ensured consistent line-height across content blocks to improve the overall layout flow.
   
   - Fixed Issue #875.
   
   ## Screenshots
   
   ### 1. The content of What is Apache Seata (fix the font-size, font-color, and typography changed to sans-serif):
   Before: 
   
   <img width="1897" height="843" alt="Screenshot (103)" src="https://github.com/user-attachments/assets/14c5ca8c-b026-4387-824e-543570e9dacb" />
   
   After:
   
   <img width="1901" height="869" alt="Screenshot (104)" src="https://github.com/user-attachments/assets/1e5ec100-ce71-4b90-9d10-907e4594c3b2" />
   
   
   ### 2. Feature List (fix the visibility and darken the text color):
   Before:
   
   <img width="1905" height="856" alt="Screenshot (105)" src="https://github.com/user-attachments/assets/61c7e96e-09be-4730-8ff2-45674ac85be8" />
   
   After:
   
   <img width="1902" height="853" alt="Screenshot (106)" src="https://github.com/user-attachments/assets/58103f4e-50cd-4bba-8911-e79bde6201b7" />
   
   
   
   ## Design Suggestion / Question
   While working on these changes, I noticed that the text in the Hero Section (the top blue header area) is very closely aligned to the left margin. To me, it feels a bit unbalanced compared to the whole home page content.
   
   Question for maintainers: Would you like me to add some padding to the left of the Hero text to center it better, or should I open a separate issue for this layout improvement?
   
   Follow this checklist to help us incorporate your contribution quickly and easily:
   
   - [x] Make sure there is a [GITHUB_issue](https://github.com/apache/incubator-seata-website/issues) filed for the change(usually before you start working on it).  Trivial changes such as typos do not necessitate a GITHUB issue. Your pull request should solely focus on addressing this specific issue without incorporating any other modifications. Each PR should resolve a single issue.
   - [x] Format the pull request title like `doc: add an operations guide document`. Each commit within the pull request should contain a meaningful subject line and body.
   - [x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
   - [x] Test your code locally by running `npm run build`, and make sure it works as expected.
   - [x] Ensure that no files under the `build` folder are included in the commit, as well as the `package-lock.json` and `yarn.lock` files.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org


Re: [PR] docs: update homepage content font-size to 18px and improve visibility [incubator-seata-website]

Posted by "funky-eyes (via GitHub)" <gi...@apache.org>.
funky-eyes merged PR #1057:
URL: https://github.com/apache/incubator-seata-website/pull/1057


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org


Re: [PR] docs: update homepage content font-size to 18px and improve visibility [incubator-seata-website]

Posted by "navjotmaan (via GitHub)" <gi...@apache.org>.
navjotmaan commented on PR #1057:
URL: https://github.com/apache/incubator-seata-website/pull/1057#issuecomment-3688470751

   I noticed the check-license is failing with a 'dependency compatibility' error (CC-BY-4.0, BlueOak, etc.). I only modified CSS and documentation text in this PR, so I believe this might be an issue with the project's current dependencies rather than my changes. Could you please advise if I need to do anything, or if this check can be bypassed?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org