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.

Sign up for GitHub

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

Jump to bottom

[Feature] Add icon with onion url hyperlink #199

Closed
MaxHillebrand opened this issue Mar 10, 2021 · 3 comments · Fixed by #348
Closed

[Feature] Add icon with onion url hyperlink #199

MaxHillebrand opened this issue Mar 10, 2021 · 3 comments · Fixed by #348
Labels
dev good first issue Good for newcomers help-wanted Issue available for takers

Comments

@MaxHillebrand
Copy link
Contributor

Currently the url is displayed as the hyperlink over the name of the merchant. Thus there is only one url field possible to fill out, usually the clearnet site is most reasonable to input.

It would be nice to have an option to add the onion url as well. This could be displayed by a hyperlinked Tor onion icon next to the name of the merchant.
@MaxHillebrand MaxHillebrand mentioned this issue Mar 10, 2021
Merged
@MaxHillebrand MaxHillebrand changed the title Add icon with onion url hyperlink [Feature] Add icon with onion url hyperlink Mar 10, 2021
@Zaxounette
Copy link
Contributor

Thanks Max for the feature request !

Will ping @dennisreimann for this, since he's the master of theses matters.

@MaxHillebrand MaxHillebrand mentioned this issue Mar 10, 2021
Closed
@dennisreimann
Copy link
Member

Yep, sounds good.

@pavlenex pavlenex added dev help-wanted Issue available for takers and removed enhancement labels Mar 23, 2022
@vaibhavgarg237 vaibhavgarg237 mentioned this issue Sep 24, 2022
Merged
@vaibhavgarg237
Copy link
Contributor

Added onion URL icon in front of titles of organizations with beautiful bounce and flash animations.

  • Sample Image for Tor-Icon

image

  • Bounce animation + Flash animation on hover over Tor-Icon Video

BTCPayServer-onion.mp4

vaibhavgarg237 added a commit to vaibhavgarg237/directory.btcpayserver.org that referenced this issue Sep 26, 2022
@vaibhavgarg237
Added onionUrl icon Close btcpayserver#199
91584e5
vaibhavgarg237 added a commit to vaibhavgarg237/directory.btcpayserver.org that referenced this issue Sep 26, 2022
@vaibhavgarg237
Added onionUrl icon Close btcpayserver#199
136ebc8
vaibhavgarg237 added a commit to vaibhavgarg237/directory.btcpayserver.org that referenced this issue Sep 27, 2022
@vaibhavgarg237
Added onionUrl icon Close btcpayserver#199
6c10792
dennisreimann pushed a commit that referenced this issue Sep 27, 2022
@vaibhavgarg237 @dennisreimann
Added onionUrl icon Close #199
33675df
Repository owner moved this from Todo to Done in Front-end dev issues to tackle 👨‍💻 Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dev good first issue Good for newcomers help-wanted Issue available for takers
Projects
No open projects
Front-end dev issues to tackle 👨‍💻
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

added onionURL icon Closes #199 vaibhavgarg237/directory.btcpayserver.org
5 participants
@dennisreimann @MaxHillebrand @pavlenex @Zaxounette @vaibhavgarg237