teambtcmap / btcmap.org

Free and open source bitcoin map web application
https://btcmap.org
GNU Affero General Public License v3.0
41 stars 10 forks source link

Consistent payment type display - Web App #15

Closed dadofsambonzuki closed 1 year ago

dadofsambonzuki commented 1 year ago

Without any specific payment tags (i.e. just currency:XBT=yes or payment:bitcoin=yes) we should show all three icons (bitcoin/on-chain, lightning and contactless bars) greyed out.

We colour them when we have a specific payment:blah=yes tag and we strike them out if we see a 'payment:blah=no' tag.

dadofsambonzuki commented 1 year ago

Looks like `payment:blah=no' doesn't remove or strike-out that payment type.

e.g. https://www.btcmap.org/map?lat=11.9675153&long=121.9274626

secondl1ght commented 1 year ago

Yes I didn't add that functionality because we don't mention adding those tags in the Wiki: https://github.com/teambtcmap/btcmap.org/wiki/Tagging-Instructions

Do you think people would explicitly add a no tag?

dadofsambonzuki commented 1 year ago

I'll update the Tagging Instructions.

We've already had feedback that this would be useful.

secondl1ght commented 1 year ago

Sounds good - will do

secondl1ght commented 1 year ago

@cogentgene could you send me strike-out versions of static/icons/ln.svg, static/icons/btc.svg and static/icons/nfc-map.svg please?

dadofsambonzuki commented 1 year ago

Hover overs should be explicit too for accepted, unknown, not accepted.

e.g. 'On-chain accepted', 'On-chain unknown', 'On-chain not accepted'.

secondl1ght commented 1 year ago

@dadofsambonzuki are you happy with the way it is now or does this still need improvement?

dadofsambonzuki commented 1 year ago

Strikethrough assets still required.

dadofsambonzuki commented 1 year ago

Paging @cogentgene - How do we feel about strikethroughs?

cogentgene commented 1 year ago

Hey, added them! Check out the Figma file. PS. if I don't respond within a few days, DM me directly in Discord and or Slack or Twitter. @dadofsambonzuki @secondl1ght

I just miss these notifications easily :( I guess being bombarded by all sorts just made me immune to checking them lol

Image

dadofsambonzuki commented 1 year ago

They are beautiful Gene.

There was talk about the onchain one being a chain. Where do you stand on that? I think I prefer the Bitcoin, although I get the logical argument against.

On Fri, 30 Sept 2022 at 12:12, Gene @.***> wrote:

Hey, added them! Check out the Figma file. PS. if I don't respond within a few days, DM me directly in Discord and or Slack or Twitter.

I just miss these notifications easily :( I guess being bombarded by all sorts just made me immune to checking them lol

[image: Image] https://user-images.githubusercontent.com/3827304/193257462-2d176098-beaa-427f-aba6-7eed9a2959c1.jpg

— Reply to this email directly, view it on GitHub https://github.com/teambtcmap/btcmap.org/issues/15#issuecomment-1263440380, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUYWY7KXWIAVSYNWQBVBEE3WA3DIJANCNFSM6AAAAAAQQE4GEE . You are receiving this because you were mentioned.Message ID: @.***>

cogentgene commented 1 year ago

@dadofsambonzuki I think I prefer the standard Bitcoin icon ₿. Chain could be mistaken for a link. I think it fairly accepted that Bitcoin symbol ₿ can mean on-chain AND lightning, UNLESS lightning is specifically mentioned with a bolt icon ⚡️ - in which case it becomes clear ₿ means on-chain. I hope..

dadofsambonzuki commented 1 year ago

Agreed.

secondl1ght commented 1 year ago

Thanks guys - this is complete now