Uh oh!
There was an error while loading. Please reload this page.
chore(docusaurus): update to latest#3014
Uh oh!
There was an error while loading. Please reload this page.
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -27,13 +27,10 @@ Thanks for your interest in contributing to Ionic's documentation! :tada: Check | ||||||
| In order to run the documentation locally, install the dependencies and run the development server: | ||||||
| ```sh | ||||||
| npm install | ||||||
Contributor There was a problem hiding this comment. The translation icon in the top menu bar appears smaller/thinner than main
ContributorAuthor There was a problem hiding this comment. I had to hide the default icon that Docusaurus renders and import a new icon via CSS. The new icon has a bolder setup. This feels a bit weird. Do we want to continue with this approach or use the default? Contributor There was a problem hiding this comment. I don't fully understand: Is the thinner icon the default or the new icon you imported via CSS? ContributorAuthor There was a problem hiding this comment. The thinner icon is the default that Docusaurus provides. Main also opts out of using the default and uses the import via CSS. The reason seems to be due to wanting a different icon at that time. Here's a screenshot of the imported icon and the default icon on main side by side.
Fast forward to today, Docusaurus has changed it's icon. It matches the one shown in main but the SVG is thinner.
I tried to match the default to main but couldn't get it so I opted to import main's icon via CSS. Realizing it now, I might be able to import the language icon from Ionicons. It might match the thickness. I'll give this a shot. liamdebeasi marked this conversation as resolved.
Uh oh!There was an error while loading. Please reload this page. | ||||||
| npm start | ||||||
Contributor There was a problem hiding this comment. The margins seem larger now: (top is branch, bottom is main. I added red guidelines for emphasis)
Also looks like the default arrow icon changed too. ContributorAuthor There was a problem hiding this comment. I think it's fixed. Let me know if it's still needs to be addressed. ContributorAuthor There was a problem hiding this comment. If you do see some differences. It's due to Docusaurus changing how they calculate the columns. | ||||||
| ``` | ||||||
Contributor There was a problem hiding this comment. The "Note" fields are no longer yellow. Is this an intentional change? Example: branch: https://ionic-docs-git-docusaurus-update-ionic1.vercel.app/docs/cli#help | ||||||
| ### Linting Documentation | ||||||
| This repository uses [Prettier](https://prettier.io/), an opinionated code formatter, in order to keep consistent formatting throughout the documentation. Run the following command to automatically fix all formatting, and then push any changes: | ||||||
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
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.
The Command+K icon in the search field has the wrong ratio:
The command symbol should roughly be the same size as the K