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

added JSON commands for picking a symbol #1936

Merged
merged 7 commits into from
Nov 14, 2023
Merged

Conversation

ArnoldSmith86
Copy link
Owner

There was a bit of discussion in Discord about this lately.

When working on the new editor, I included Noto Emoji for safe cross-platform display of emoji and all of the game-icons.net icons to the internal VTT assets. I also made a nice icon picker for them but so far it was only accessible from the game metadata.

This PR adds two JSON editor commands. One for image-based icons anywhere where the Upload Asset command was available. And one for buttons with class symbols or material-icons for the text property.

My plan is still to have new properties and card object types that make this even easier but this is a nice first easy step that makes these resources more freely available.

This is also still missing proper attribution. For now you will have to insert the proper attribution for game-icons.net icons into the game metadata manually.

@ArnoldSmith86 ArnoldSmith86 added enhancement New feature or request editor labels Nov 12, 2023
@ArnoldSmith86
Copy link
Owner Author

PR-SERVER-BOT: You can play around with it here: https://test.virtualtabletop.io/PR-1936/pr-test (or any other room on that server)

After merging, a backup will be available at https://beta.virtualtabletop.io/editor/PR1936-pr-test.

@96LawDawg
Copy link
Collaborator

Kinda glitchy as the images are loading. Will it take a long time every time?

PR1936.mp4

@96LawDawg
Copy link
Collaborator

Quite a few VTT symbols are missing. For example, all of the arrows, the seats/chairs. But many other things also.

@ArnoldSmith86
Copy link
Owner Author

It's probably missing the ones that are just copied from Material Icons.

Do the glitches happen in production as well? What about the other PR? Which browser?

@96LawDawg
Copy link
Collaborator

96LawDawg commented Nov 12, 2023

What about the other PR?

Same problem there.

@96LawDawg
Copy link
Collaborator

Strange. I posted an answer to your other questions. They show up in Discord but not here. Will post again, just in case.

Do the glitches happen in production as well?

No. The icon picker for meta data has always worked fine.

What about the other PR?

Same problem.

Which browser?

Browsers with problem are Crome and Edge in Windows 11. No problem in Firefox.

@ArnoldSmith86
Copy link
Owner Author

The icon picker for meta data has always worked fine.

Is it still working in this PR? The only difference should be the size of the icons. I could show the small previews and only show bigger ones when filtered to <100.

@96LawDawg
Copy link
Collaborator

Is it still working in this PR?

Yes, except for the flags.

@ArnoldSmith86
Copy link
Owner Author

Is it still working in this PR?

Yes, except for the flags.

Those are also broken in production though, right? Windows doesn't have the flags iirc? I can probably make an exception for those and always display the images instead.

@96LawDawg
Copy link
Collaborator

Yes, except for the flags.

Those are also broken in production though, right?

Yes. They show up when you mouse over them, but otherwise just have the letters.

@ArnoldSmith86
Copy link
Owner Author

How about now?

@96LawDawg
Copy link
Collaborator

Works in Chromium and Firefox on Windows 11. Does not work on iOS on newish iPad.

Copy link
Collaborator

@96LawDawg 96LawDawg left a comment

Choose a reason for hiding this comment

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

Approved through test

@ArnoldSmith86 ArnoldSmith86 enabled auto-merge (squash) November 14, 2023 09:22
@ArnoldSmith86 ArnoldSmith86 merged commit dd3da17 into main Nov 14, 2023
6 checks passed
@ArnoldSmith86 ArnoldSmith86 deleted the feature-symbolPickerAsset branch November 14, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants