Skip to content

Make all references to geometric points conform to standards #30

Open

Description

@skytreader

See: https://github.com/skytreader/PyGame-Objects/wiki/Standards-and-Conventions

Make it so that all tuples describing positions describe them as (row, col). draw_offset currently uses (width, height) and is confusing.

Activity

  1. skytreader commented on Feb 27, 2018

    @skytreader
    OwnerAuthor

    On the other hand, check with pygame conventions and conform to that: https://www.pygame.org/docs/ref/draw.html#pygame.draw.line

    Looks like pygame.draw.line expects points in the actual (x, y) format; column indices go first before row indices.

  2. added a commit that references this issue on Mar 15, 2018
  3. skytreader commented on Mar 17, 2018

    @skytreader
    OwnerAuthor

    More notes on PyGame conventions: font.size returns (width, height).

  4. changed the title [-]draw_offset is inconsistent[/-] [+]Make all references to geometric points conform to standards[/+] on Mar 24, 2018
  5. skytreader commented on Apr 11, 2018

    @skytreader
    OwnerAuthor

    Violation: Docstring for Drawable constructor says that draw_offset is a (width, height) tuple but GameScreen._is_drawable_clicked uses index 1 for width and index 0 for height.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions