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

Centralize resource access and add Rods icons #3774

Merged
merged 17 commits into from
Dec 12, 2022

Conversation

thelsing
Copy link
Collaborator

@thelsing thelsing commented Dec 11, 2022

Identify the Bug or Feature request

Description of the Change

  • centralise acces to all ressource via a RessourceManager class. This caches created icons and images.
  • remove image references from abaile xml files and set those in code
  • add enums for all used images, icons and all borders
  • move all "classic" images to net/rptools/maptool/client/images
  • add Rod Takeharas icons to net/rptools/maptool/client/icons/rod_takehara
  • add preference to chose an icon theme
    Some icons are missing a new version atm. For those icons the old version is used.

Possible Drawbacks

I might have messed up somewhere when moving around images. If so we will git exceptions on opening dialogs or red squares for the missing images. Both is easy to spot.

Release Notes

  • add icon theme support and modern icons by Rod Takehara

This change is Reviewable

# Conflicts:
#	src/main/resources/net/rptools/maptool/client/ui/forms/tokenPropertiesDialog.xml
#	src/main/resources/net/rptools/maptool/language/i18n.properties
#	src/main/resources/net/rptools/maptool/language/i18n_cs.properties
#	src/main/resources/net/rptools/maptool/language/i18n_da.properties
#	src/main/resources/net/rptools/maptool/language/i18n_de.properties
#	src/main/resources/net/rptools/maptool/language/i18n_es.properties
#	src/main/resources/net/rptools/maptool/language/i18n_fr.properties
#	src/main/resources/net/rptools/maptool/language/i18n_it.properties
#	src/main/resources/net/rptools/maptool/language/i18n_ja.properties
#	src/main/resources/net/rptools/maptool/language/i18n_nl.properties
#	src/main/resources/net/rptools/maptool/language/i18n_pl.properties
#	src/main/resources/net/rptools/maptool/language/i18n_pt.properties
#	src/main/resources/net/rptools/maptool/language/i18n_ru.properties
#	src/main/resources/net/rptools/maptool/language/i18n_sv.properties
#	src/main/resources/net/rptools/maptool/language/i18n_uk.properties
#	src/main/resources/net/rptools/maptool/language/i18n_zh.properties
@Phergus Phergus changed the title Centralise ressource acces and add Rods icons Centralize resource access and add Rods icons Dec 12, 2022
@Phergus Phergus merged commit 732798e into RPTools:develop Dec 12, 2022
@thelsing thelsing deleted the feature-icons branch December 12, 2022 22:26
@thelsing thelsing mentioned this pull request Dec 15, 2022
@cwisniew cwisniew added the feature Adding functionality that adds value label Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding functionality that adds value
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants