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

Changed color of content text (not title & subtitle) for blogs and ad… #6683

Merged
merged 6 commits into from
Aug 17, 2023

Conversation

seanzboland
Copy link
Contributor

@seanzboland seanzboland commented Aug 11, 2023

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.

πŸ‘‡ __ Let's make a quick check before the contribution.

PR Type

  • Bugfix
  • Feature
  • Refactoring

Context

Before submitting pull request, please make sure:

  • My contribution builds clean without any errors or warnings
  • I've merged recent default branch -- main and I've no conflicts
  • I've tried to respect high code quality standards
  • I've didn't break any original functionality

Optional

  • I've tested it at </ksm/collection>
  • I've tested PR on mobile
  • I've written unit tests πŸ§ͺ
  • I've found edge cases

Did your issue had any of the "$" label on it?

  • Fill up your DOT address: Payout

Community participation

Screenshot πŸ“Έ

  • My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

πŸ€– Generated by Copilot at 79f61ff

This pull request improves the readability and appearance of the blog posts by tweaking the color of some elements and the spacing of the headings in the .markdown-body class. It modifies the k-accentlight2-dark variable in _theme.scss and adds some CSS rules to BlogPost.vue.

πŸ€– Generated by Copilot at 79f61ff

markdown-body styled
More space for headings, code
A breath of spring air

…ded more margin between paragraphs in content
@seanzboland seanzboland requested a review from a team as a code owner August 11, 2023 18:33
@seanzboland seanzboland requested review from roiLeo and vikiival and removed request for a team August 11, 2023 18:33
@netlify
Copy link

netlify bot commented Aug 11, 2023

βœ… Deploy Preview for koda-canary ready!

Name Link
πŸ”¨ Latest commit 19be55c
πŸ” Latest deploy log https://app.netlify.com/sites/koda-canary/deploys/64dddbac7da33600087ab12f
😎 Deploy Preview https://deploy-preview-6683--koda-canary.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@reviewpad reviewpad bot added small Pull request is small waiting-for-review labels Aug 11, 2023
@reviewpad
Copy link
Contributor

reviewpad bot commented Aug 11, 2023

AI-Generated Summary: This pull request introduces visual adjustments to the blog post layout in the application. Specifically, it adds more spacing between headers (h1, h2, h3, h4) from the second child onwards in blog posts, introducing a larger gap of 3rem for better readability and structure. Also, it changes the color of the content text (excluding headers) in the theme's SCSS file. It modifies the color values assigned to 'k-accentlight2-dark': the color goes from '#363234' to '#191718' for the first theme and from '#ffffff' to '#b9b9b9' for the second. These changes should improve the aesthetic look and feel of blog posts in the application.

@exezbcz
Copy link
Member

exezbcz commented Aug 11, 2023

12px here
image

  • I cant see colors changed

@seanzboland
Copy link
Contributor Author

12px here image

  • I cant see colors changed

@exezbcz
The text color is changed from #ffffff into #b9b9b9

Screen Shot 2023-08-11 at 3 15 26 PM
Screen Shot 2023-08-11 at 3 15 40 PM

@seanzboland
Copy link
Contributor Author

@exezbcz
Do you mean 12px for the margin?
It was asked to add more margin on top of each paragraph (between paragraph title and its previous paragraph)

@exezbcz
Copy link
Member

exezbcz commented Aug 11, 2023

@seanzboland
image

@seanzboland
Copy link
Contributor Author

@exezbcz
Please confirm if I am thinking correctly.

  • I will keep color for title of each paragraph, and try to change only the color of content
  • Can I add a new token for dark/light mode for this blog text color?
    Actually, k-accentlight2dark is only used for blog text color, and that's why I wanted to change it.

@yangwao
Copy link
Member

yangwao commented Aug 12, 2023

@seanzboland please use Pull Template otherwise I'm closing your PR.

@exezbcz
Copy link
Member

exezbcz commented Aug 14, 2023

@seanzboland

  • I will keep color for title of each paragraph, and try to change only the color of content

yes, color of heading for darkmode

  • Can I add a new token for dark/light mode for this blog text color?
    Actually, k-accentlight2dark is only used for blog text color, and that's why I wanted to change it.

not sure, you would have to ask devs

@yangwao
Copy link
Member

yangwao commented Aug 16, 2023

how they are here 4 participants πŸ‘€
image

@exezbcz
Copy link
Member

exezbcz commented Aug 16, 2023

@seanzboland any updates sir?

@seanzboland
Copy link
Contributor Author

seanzboland commented Aug 16, 2023 via email

@exezbcz
Copy link
Member

exezbcz commented Aug 16, 2023

@seanzboland If you want, you can still work on this issue. So that you know, there's a risk someone else could be assigned and deliver earlier.

Copy link
Contributor

@roiLeo roiLeo left a comment

Choose a reason for hiding this comment

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

βœ… code lgtm

@roiLeo roiLeo added the S-code-lgtm-βœ… code review guild has reviewed this PR and it's code is approved label Aug 17, 2023
@seanzboland
Copy link
Contributor Author

Merging can be performed automatically with 2 approving reviews.

@roiLeo
I have 1 approval now, and would like to hear the following steps.

@seanzboland
Copy link
Contributor Author

@exezbcz
I would like to continue work on ticket, and updated regarding your comments.
Please check it.

@seanzboland
Copy link
Contributor Author

Screen Shot 2023-08-17 at 3 18 04 AM

Copy link
Member

@vikiival vikiival left a comment

Choose a reason for hiding this comment

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

Code oki

@exezbcz pls confirm if its visually oki

@exezbcz
Copy link
Member

exezbcz commented Aug 17, 2023

image

  • please change the color of subheading as well in dark mode to the paragraph text

this should be 40, not 48px
image

  • this should be 12px
    image

light mode

image

  • same colors as here (k-darkmode), use it for subheading please as well
  • here:
    image
    h1 stays true black, that is correct

Otherwise oki!

@seanzboland
Copy link
Contributor Author

@exezbcz
I changed colour of subtitle and fixed the margin

@exezbcz
Copy link
Member

exezbcz commented Aug 17, 2023

smol detail
in light mode the h2 has different color from the p text
image
image

should be this one: #191718

@seanzboland
Copy link
Contributor Author

@exezbcz
I fixed it.

@codeclimate
Copy link

codeclimate bot commented Aug 17, 2023

Code Climate has analyzed commit 19be55c and detected 0 issues on this pull request.

View more on Code Climate.

@sonarcloud
Copy link

sonarcloud bot commented Aug 17, 2023

Kudos, SonarCloud Quality Gate passed!Β  Β  Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@yangwao
Copy link
Member

yangwao commented Aug 17, 2023

pay 30 usd

@yangwao yangwao merged commit a4d1629 into kodadot:main Aug 17, 2023
14 checks passed
@yangwao
Copy link
Member

yangwao commented Aug 17, 2023

😍 Perfect, I’ve sent the payout
πŸ’΅ $30 @ 4.75 USD/DOT ~ 6.316 $DOT
πŸ§— 12AiUwY6xy6jmKQELdxWPSKacd5tqXBYjcq3p1gx63Mhmcgz
πŸ”— 0x53d39181a4fccb4ec0882fb3f5ae2f4a41a1fd3f1dfeb52072c6de5e19d7e7cd

πŸͺ… Let’s grab another issue and get rewarded!
πŸͺ„ github.com/kodadot/nft-gallery/issues

@yangwao yangwao added the paid pull-request has been paid label Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paid pull-request has been paid S-code-lgtm-βœ… code review guild has reviewed this PR and it's code is approved S-visual-ok-βœ… small Pull request is small waiting-for-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blog article styling
6 participants