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

Task 25 - Help/Wiki Viewer #65

Open
wants to merge 35 commits into
base: devel
Choose a base branch
from

Conversation

ponbac
Copy link

@ponbac ponbac commented Dec 6, 2021

Adds a button in the bottom left of the window, which on click opens this window:

Markdown formatting logic is inside of _bbcode_to_markdown() in HelpViewer.gd.

Alternate solution without _fetch_wiki_titles() possible if you manually create a list of titles to download inside of _download_wiki_pages.

ponbac and others added 30 commits October 26, 2021 14:17
Copy link
Collaborator

@RuthgerD RuthgerD left a comment

Choose a reason for hiding this comment

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

Quite simple code for a great addition

project/Entry.gd Outdated Show resolved Hide resolved
project/src/ui/help_viewer/HelpViewer.gd Outdated Show resolved Hide resolved
project/src/ui/help_viewer/HelpViewer.gd Outdated Show resolved Hide resolved
project/src/ui/help_viewer/HelpViewer.gd Outdated Show resolved Hide resolved
project/src/ui/help_viewer/HelpViewer.gd Outdated Show resolved Hide resolved
project/src/ui/help_viewer/HelpViewer.gd Show resolved Hide resolved
project/src/ui/help_viewer/HelpViewer.gd Show resolved Hide resolved
@RuthgerD RuthgerD added the accepted accepted but can't be merged yet label Dec 12, 2021
@RuthgerD
Copy link
Collaborator

Looks good, couple of rough edges but since its well self contained no reason to block this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted accepted but can't be merged yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants