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

CC: Expand on make_gui docstring #4449

Merged
merged 3 commits into from
Jan 26, 2025
Merged

Conversation

qwint
Copy link
Collaborator

@qwint qwint commented Jan 8, 2025

What is this fixing or adding?

adds docstring to make_gui describing what things you might want to change without dealing with kivy/kvui directly (there are better places to document those)

How was this tested?

eyeballs

If this makes graphical changes, please attach screenshots.

Verified

This commit was signed with the committer’s verified signature.
shockey kyle shockey
…hange without dealing with kivy/kvui directly (there are better places to document those)
@github-actions github-actions bot added affects: core Issues/PRs that touch core and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jan 8, 2025
qwint and others added 2 commits January 9, 2025 10:04

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Doug Hoskisson <beauxq@users.noreply.github.com>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Doug Hoskisson <beauxq@users.noreply.github.com>
Copy link
Collaborator

@beauxq beauxq left a comment

Choose a reason for hiding this comment

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

eyeballs also

@ScipioWright ScipioWright added the is: enhancement Issues requesting new features or pull requests implementing new features. label Jan 10, 2025
@qwint qwint added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jan 23, 2025
Copy link
Member

@black-sliver black-sliver left a comment

Choose a reason for hiding this comment

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

I think this should be moved to a separate doc at some point, that properly explains how to write a client and what kvui.GameManager is, but it's probably fine until we have that.

@black-sliver black-sliver merged commit 90417e0 into ArchipelagoMW:main Jan 26, 2025
16 checks passed
@qwint
Copy link
Collaborator Author

qwint commented Jan 26, 2025

I explicitly only mentioned those two usecases because they can be done without greater kivy knowledge,
a full kivy primer would be helpful elsewhere I agree (I might see how much I can hash out myself but I wouldn't count on it)

@qwint qwint deleted the cc_doc_logging branch January 26, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: core Issues/PRs that touch core and may need additional validation. is: enhancement Issues requesting new features or pull requests implementing new features. waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants