-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
chore(docusaurus): update to latest #3014
Changes from 51 commits
9cfc0b6
78c3bb0
806ced2
e90ea66
da99aa3
fb2e419
c969de4
5af40a9
3b4173d
1b4b665
7f89e7b
163f9f1
0b3cbdb
b6381b0
af1ec1f
487d8a2
782a7ae
ae81a73
e4af97a
8eaf36c
ad56f82
e5d9ac8
5209546
6ba7427
44eb7cc
642aaea
26d28bc
eaefb54
3a3e80c
5e2ab7e
07afdfa
d1ec4be
28d7b26
91c7b18
55bc40a
2e051f5
4ac4192
91db9ca
fb90efd
a3fe039
ac11a79
70dd033
5c0c6e0
6473b5d
3b00897
7f04586
bc3f032
0bd636a
f05346a
f74a89b
449c02a
e059134
72a0ccf
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
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 --legacy-peer-deps | ||
$ npm start | ||
npm install | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't fully understand: Is the thinner icon the default or the new icon you imported via CSS? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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. (main on left, Docusaurus on right) Fast forward to today, Docusaurus has changed it's icon. It matches the one shown in main but the SVG is thinner. (main on left, Docusaurus on right) 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.
Show resolved
Hide resolved
|
||
npm start | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it's fixed. Let me know if it's still needs to be addressed. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If you do see some differences. It's due to Docusaurus changing how they calculate the columns. |
||
``` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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 |
||
|
||
> [!NOTE] | ||
> Certain versions of npm (5-8) and Node.js (10-16) are required to run certain scripts. | ||
|
||
### 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: | ||
|
This file was deleted.
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