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

services.icon is documented, but not working #5984

Open
tadast opened this issue Nov 29, 2023 · 1 comment
Open

services.icon is documented, but not working #5984

tadast opened this issue Nov 29, 2023 · 1 comment

Comments

@tadast
Copy link

tadast commented Nov 29, 2023

Docs suggest a custom SVG can be used in the sidebar, but it doesn't appear to be working and I can't seem to find any code that is using this option.

Perhaps it needs to be removed from the docs if the option is not supported?

@robertknight
Copy link
Member

robertknight commented Nov 29, 2023

The short answer is that this setting still exists but is not effective any more, and we should indeed remove it from the documentation.

This setting is still used in the code, but you'd need to use something like VS Code which understands TypeScript projects to find all usages of the icon field.

To go into more detail, the option was originally added for publishers like eLife who have a white-labeled version of Hypothesis on their site. We subsequently added the concept of "organizations" to Hypothesis which can have their own logo, and that takes priority over the services.icon field. We later made it so that all groups are assigned to an organization, using a default one with a Hypothesis logo if they haven't been assigned to a more specific one. I believe this means, although it needs to be confirmed, that the service.icon value is never used.

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

No branches or pull requests

2 participants