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

Page "Creating a Demo" - First time impressions #33

Open
doglman opened this issue Jul 17, 2023 · 2 comments
Open

Page "Creating a Demo" - First time impressions #33

doglman opened this issue Jul 17, 2023 · 2 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@doglman
Copy link

doglman commented Jul 17, 2023

During my first time using the SSS, I attempted to go through the "Creating a Demo" instructions from the documentation. My goal was to create a simple "Hello World" demo that displayed my desired text on the screen.

Here are some of the issues I encountered while going through this process:

Public-facing links to internal documentation

In the section screen (link), there is a brief paragraph describing a self.screen object, which "controls what will be drawn on the screen". It contains a link to an "API doc", which is a Notion page. This Notion page is not available to the public.

Suggestion: Fix this link. The page containing equivalent content to what is on our Notion page can be found in Tutorials > Game Display

Duplication of API content

The Notion page didn't have any specifics describing what the input parameters were for each function... which was not helpful for me, who wanted to actually use these functions instead of just get a high-level understanding of what they did.

I found equivalent Display documentation that did have all of the function explanations.

Suggestion: Merge these two sets of documentation into a single source that provides high-level explanations and the specific parameter details.

Inconsistent naming in the documentation

In the "Creating a demo" Tutorial mentioned earlier, the term used to describe the object that functions like draw_hline() and draw_text() can be called upon is screen.

But in this page, it is referred to as a Game Display.

And in the Display documentation, it is referred to as a Display.

As a new user, this added confusion and consumed time for me to figure out that all three terms referred to the same concept.

Suggestion: Adopt a consistent naming scheme across the entire documentation. Refer to this object as Display, since that is what the code uses.

Summary:

There is some inconsistency in naming scheme and duplication of documentation for a vital object. These issues can be resolved by merging all the documentation into a single location, using a single naming scheme.

@doglman doglman added the documentation Improvements or additions to documentation label Jul 17, 2023
christopolise added a commit that referenced this issue Aug 24, 2023
@christopolise
Copy link
Contributor

The link pointing to internal documentation has been fixed in 9c7a565

@christopolise
Copy link
Contributor

Inconsistent naming in the documentation:

@apal6981 is the author of the Game Display document and the Display class. Could you take a look and make some quick changes on the docs branch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants