-
Notifications
You must be signed in to change notification settings - Fork 71
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
"Mirador" tag is added independently of Islandora Mirador being enabled #1608
Comments
@mjordan This could be moved to Islandora Mirador. On canterburystories.nz we have several custom modules adding display hints like Carousel, or Slideshow, leaving Islandora Defaults untouched but migrating terms to |
Ah ok. Like a migration in islandora_mirador that adds the Mirador term to the islandora display hints vocabulary? |
Not necessarily a migration. You can add it in an install/update hook. |
Right. I’ll try to get a PR up today. |
Islandora Defaults add the "Mirador" display hint tag (https://github.com/Islandora/islandora_defaults/blob/8.x-1.x/migrate/tags.csv#L4), regardless of whether Islandora Mirador is enabled (it isn't by default). This means that the "Mirador" display hint is exposed to the user even when that module is not enabled, which I believe is confusing:
Couple of possible fixes:
Other suggestions?
The text was updated successfully, but these errors were encountered: