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
Updates to Help #1259
Updates to Help #1259
Conversation
I think files in You can eventually remove them (in separate commit) to see if build and everything else goes well. |
You can also rename Though, please do that in separate commit. And don't forget to fix also references to that file in |
Src/Localization/English/Main.html
Outdated
<br> | ||
<h1>Components</h1><br> | ||
|
||
<h2>Components</h2> | ||
Open-Shell has three major components:<ul> | ||
<li><a href="Menu.html">Open-Shell Menu</a></li> |
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.
Open-Shell Menu
looks weird here (other components don't have Open-Shell
in name).
I'd suggest to change it to Start menu
.
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.
Or maybe Classic Start Menu
(as other components have Classic
too).
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.
I agree that it looks weird, but I personally think it makes more sense to keep it consistent with the product name (as shown as the titles of the other pages, etc.) Maybe it could be a future renaming effort?
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.
Oki, lets keep it as it is for now.
I will look at renaming the Menu.html component. Do you know of the other references off-hand?
So I can remove the entire localization folder without issue? |
I have found it in
I believe so. Because I don't see any references to that folder. |
Hmm, not sure what's the issue with help compilation :( |
I think I missed some references to StartMenu.html. I'll try to fix. |
Ok, seems like it may be fixed now. Would you like me to revert the BuildInstaller change? Edit: I also still need to attach one of the related issues. |
Perfect. And the help now works nicely 👍
Yes, please. |
Could you also fix other languages in And for the So that we will know why it was removed in the future. |
This should fix the initial error message upon launching the help dialog. Fixes #524 fix - Rename Menu.html in other languages
Folder is not used anywhere in build, instead Src/Localization is used.
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.
Looks really great.
Thank you so much.
@bonzibudd |
@ge0rdi I'll also provide some new images for the |
Fixes #524, fixes #880, and fixes #1173