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

(Prototype) Alternate hexmap implementation #100

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

michaeltlombardi
Copy link
Contributor

This change builds on the prototype by @cyberselkie to emit a hexmap on a page. It emits a list of buttons above a shoelace tab group. The buttons should display as hex map tiles. Clicking the buttons selects the appropriate tab in the group. Eventually, selecting a tab should also highlight/select the hex in the map above.

This change uses AlpineJS for the button action, injecting the x-ref attribute into the sl-tabs-group so the buttons can call the show() method on the group instead of requiring an external script or module.

None of the styling is in place. This PR is intended for comparison only, the canonical PR should be the original submission with updates and this PR and commit should be thrown away at that time.

Charlotte and others added 5 commits October 2, 2023 19:45
This change builds on the prototype by @cyberselkie to
emit a hexmap on a page. It emits a list of buttons
above a shoelace tab group. The buttons should display
as hex map tiles. Clicking the buttons selects the
appropriate tab in the group. Eventually, selecting
a tab should also highlight/select the hex in the
map above.

None of the styling is in place. This PR is intended
for comparison only, the canonical PR should be the
original submission with updates and this PR and
commit should be thrown away at that time.
Copy link

netlify bot commented Nov 11, 2023

Deploy Preview for platen ready!

Name Link
🔨 Latest commit d3ebd1f
🔍 Latest deploy log https://app.netlify.com/sites/platen/deploys/654fe804de39b6000891a019
😎 Deploy Preview https://deploy-preview-100--platen.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.

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