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

Create usage example for draw_line_on_window #235

Open
wants to merge 3 commits into
base: usage-examples
Choose a base branch
from

Conversation

SimonRhook
Copy link

@SimonRhook SimonRhook commented Nov 12, 2024

Overview

Added usage example for draw_line_on_window including C# OOP and Top level, C++ and Python

Files Included

Splashkit Function: draw_line_on_window

  • Title and explanation (.txt)
  • C++ code
  • C# code (top-level statements)
  • C# code (Object-Oriented Programming)
  • Python code
  • Screenshot

Usage Example Checks

  • Simple, clear demonstration of the function
  • Code uses Splashkit functions
  • Tested in Chrome and Firefox

Copy link

netlify bot commented Nov 12, 2024

Deploy Preview for splashkit-io ready!

Name Link
🔨 Latest commit 3e4d1e8
🔍 Latest deploy log https://app.netlify.com/sites/splashkit-io/deploys/6732f1de72c0f000086ff4de
😎 Deploy Preview https://deploy-preview-235--splashkit-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

public static void Main()
{
// Create Window
Window start = new Window("draw_line_on_window", 600, 600);

Choose a reason for hiding this comment

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

I would suggest changing the name of this window to be something different. On the website preview this line comes up as the "function" highlighted. because it has the same syntax. A simple fix would be to just remove the underscore. Make sure you change this over all of the program files so they remain the same.

Example of what im meaning -
image

Otherwise, ive reviewed all the code files and they run correctly. I'm happy to approve this once that change is done.

Copy link
Author

@SimonRhook SimonRhook Nov 12, 2024

Choose a reason for hiding this comment

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

This change has now been implemented

Copy link

@breezy-codes breezy-codes left a comment

Choose a reason for hiding this comment

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

I've reviewed this new usage example and all codes run correctly free of any errors.
The code comments match over all files and the structure is well formed. This also a great demonstration of draw line on window.

  • All required files are present.
    • Title and explanation (.txt)
    • C++ code
    • C# code (top-level statements)
    • C# code (Object-Oriented Programming)
    • Python code
  • Code correctly uses Splashkit functions.
  • Code clearly demonstrates the function.
  • All versions maintain the same structure and comments.
  • C++ code ran correctly.
  • C# top level code ran correctly.
  • C# OOP code ran correctly.
  • Python code ran correctly.

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

Successfully merging this pull request may close these issues.

2 participants