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

[FLORA-198] Fix mismatching OpenSearch names #427

Conversation

iceCream-Taco
Copy link
Contributor

@iceCream-Taco iceCream-Taco commented Sep 11, 2023

Proposed changes

Match the "ShortName" field in the OpenSearch XML file with the "title" property in the link to the OpenSearch XML document in the head element. This complies with Mozilla's OpenSearch description format page and results in the "add search engine" button disappearing as it should once Flora has been added.

Also, the OpenSearch description does not make any sense as it now matches the short name, whereas it should describe the search engine (Flora). As it is not relevant I've left it for now, but it should probably be changed to something more appropriate in the future.

Contributor checklist

@tchoutri
Copy link
Contributor

@iceCream-Taco Hi, and thank you for this contribution, it really bugged me. :)
From what I understand, both fields should only read "Flora", then?

@iceCream-Taco
Copy link
Contributor Author

iceCream-Taco commented Sep 12, 2023

@tchoutri I did consider changing both to Flora, but the search title examples are "Search MDC" and "Yahoo! Search" on the MDN page, and as the title field already read "Search On Flora" I changed the ShortName field to match this instead. Looking around though it does look like most webpages stick with the short name (Wikipedia, Google Cloud, Bing etc.) so I'll change the name/title to "Flora" to match them, as you suggested.

@tchoutri
Copy link
Contributor

Fantastic. Thanks a lot!

@tchoutri tchoutri merged commit 92c9d3d into flora-pm:development Sep 12, 2023
mau5mat pushed a commit to mau5mat/flora-server that referenced this pull request Sep 14, 2023
* [FLORA-198] Match OpenSearch name in head and XML
RaoulHC pushed a commit to RaoulHC/flora-server that referenced this pull request Oct 18, 2023
* [FLORA-198] Match OpenSearch name in head and XML
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.

2 participants