-
Notifications
You must be signed in to change notification settings - Fork 48
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
Make blockquotes more pretty and make citation more clear #2
Conversation
layouts/partials/head.html
Outdated
@@ -17,7 +17,11 @@ | |||
<link rel="icon" href="{{ `images/favicon.png` | absURL }}" type="image/x-icon"> | |||
|
|||
{{ "<!-- fonts -->" | safeHTML }} | |||
<link href="https://fonts.googleapis.com/css?family={{replace .Site.Params.font_family ' ' '+' | title}}:300,400,700&display=swap" rel="stylesheet"> | |||
<link rel="preconnect" href="https://fonts.gstatic.com"> | |||
<link href="https://fonts.googleapis.com/css2?family={{replace .Site.Params.font_family_a ' ' '+' | title}}+{{replace .Site.Params.font_family_b ' ' '+' | title}}:wght@300;400;700&display=swap" rel="stylesheet"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I am curious. Why was this change necessary? Not sure I follow your path of thinking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remember
Hi @Idadelveloper, this looks great and I am almost ready to merge this! The only thing that is a red flag to me for this Pull Request is the way fonts are loaded into the theme from the config. It is possible there is a bug here you caught working on this, but if it is a bug, we should look at fixing this another way that does not depend on the config file values.
Does this make sense? Let me know if you have doubts. Thanks again for working on this! 🙌🏻
@jwflory there's definitely a bug. Will get rid of this change together with the changes in the config file.
layouts/partials/head.html
Outdated
@@ -17,7 +17,11 @@ | |||
<link rel="icon" href="{{ `images/favicon.png` | absURL }}" type="image/x-icon"> | |||
|
|||
{{ "<!-- fonts -->" | safeHTML }} | |||
<link href="https://fonts.googleapis.com/css?family={{replace .Site.Params.font_family ' ' '+' | title}}:300,400,700&display=swap" rel="stylesheet"> | |||
<link rel="preconnect" href="https://fonts.gstatic.com"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a performance gain for loading font caches or something earlier? Sorry if it is an obvious question, I am curious. 🙊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jwflory really has nothing much to do with performance. So when "Open+Sans" is used it is not being parsed correctly therefore an error shows in the console.
I just removed the changes I made in this file and the config file. We could figure out a better way to tackle this :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Idadelveloper Good catch here. I didn't notice this bug until now. You could open an issue on unicef/inventory so we could talk through how best to fix that one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Idadelveloper, this looks great and I am almost ready to merge this! The only thing that is a red flag to me for this Pull Request is the way fonts are loaded into the theme from the config. It is possible there is a bug here you caught working on this, but if it is a bug, we should look at fixing this another way that does not depend on the config file values.
Does this make sense? Let me know if you have doubts. Thanks again for working on this! 🙌🏻
This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent. Looks good to me! Going to merge this and update the git submodule in the upstream inventory repo shortly.
Merging! 🎬
This incorporates work from PR unicef/inventory-hugo-theme#2 by @Idadelveloper. It pulls commits to make the blockquotes more visually appealing. And it supports images, nice! This likely needs corresponding documentation, but will revisit this farther down the line. Contributes to #14 but does not close it. PR #23 will require rebasing for a passing CI build. Signed-off-by: Justin W. Flory (he/him) [UNICEF Innovation] <jflory@unicef.org>
This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com>
* assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
Pull Requests #2 and #8. Corresponds to unicef/inventory#14. This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear. Signed-off-by: Ida Delphine <idadelm@gmail.com> Show big quotes for citation (#8) * assets: Enhance look of blockquotes and citation (#2) This commit tackles the issue in the inventory repo on making blockquotes more pretty and citation clear (unicef/inventory#14) Signed-off-by: Ida Delphine <idadelm@gmail.com> * layouts: Add big quotes to citation Signed-off-by: Ida Delphine <mida@unicef.org>
@jwflory this PR handles the styling for issue #14 in the inventory repo.
PS: My vs code formatter kinda made changes throughout the css file. The changes I really made are from line 594.