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

Client rewrite #21

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Client rewrite #21

wants to merge 7 commits into from

Conversation

AphonicChaos
Copy link
Owner

@AphonicChaos AphonicChaos commented Apr 16, 2023

TL;DR

This PR is a completely rewrite of the client code, though I anticipate that I'll back-port the authentication code from the other branch. It's currently a draft, which is why the warwomb-client-old directory exists, rather than being deleted outright.

TODO

  • add a "surface" behind the left and right player token displays, as well as the "action" area on top so text and other content is easier to see
  • provide some means of collapsing/subduing those sections so they aren't distracting during play
  • back-port the Auth0 authentication code
  • re-implement draggable tokens
  • separate display into two layers:
    • "hud" layer where ui elements that aren't meant to scale or be dragged around can go
    • "interactive" layer where the map itself can be zoomed and panned, scaling elements with it, and elements can be marked as movable, having menus, etc
  • allow the map to be scaled/rotated like before

Note that adding the layer is all I intend to do before merging -- making them interactive is a different story

Screenshots

Obviously, from the after you can tell this is still work-in-progress.

Before

2023-04-16-100841_2917x1476_scrot

After

2023-06-10-201736_1440x1177_scrot

@AphonicChaos AphonicChaos marked this pull request as draft April 16, 2023 17:19
@AphonicChaos
Copy link
Owner Author

After drawing the player areas onto a surface, text is legible, but it's also even more apparent that I need to work on splitting those "layers", and likely move away from CSS grid to give me more control over adding and such of the grid items (which is why everything looks so squished in the screenshot).

…d them being on this branch just clutters things up
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.

1 participant