hasadna / open-bus-map-search

open-bus-map-search
https://open-bus-map-search.hasadna.org.il/dashboard
MIT License
48 stars 85 forks source link

There are two donate buttons at the navbar which show different things #753

Open VSciFlight opened 1 month ago

VSciFlight commented 1 month ago

Noticed on the live website of the project right here

These are the buttons I am speaking of. image

link: https://open-bus-map-search.hasadna.org.il/dashboard

The right donation button opens a modal which explains how to donate where the left donation button opens the donation page on jgive.co.il. Both of them show "לתרומות" tooltip.

Donation modal when clicking on the right donation button: image

JGive website when clicking on the left donation button: https://www.jgive.com/new/he/ils/donation-targets/3268#donation-modal

again, it was noticed on the live website of the project right here: https://open-bus-map-search.hasadna.org.il/dashboard

NoamGaash commented 1 month ago

Thank you for opening this issue! The modal is the better version. So despite both links are valid, converting the sidebar link to open the modal could be great

iDanVas commented 1 month ago

I'll take it if that's ok. I`m from Maakaf group.

NoamGaash commented 1 month ago

Thank you @iDanVas !

:rocket:

iDanVas commented 1 month ago

Thank you @iDanVas !

🚀

I have though how to change the donation button and came up with 3 options.

  1. Remove the donation link from the side bar and clone the donation popup below the links.
  2. Same as the 1st option but to add it to the links with onclick event that will open the donation window.
  3. Change the donation pop-up to page and inside it will be the internal donation and the link to the external donation.

Which one do you prefer?

VSciFlight commented 1 month ago

I think the donation itself should lay within the modal. In terms of UI/UX the user stays within the website and doesn't "wander" outside of it. Eases the flow for the user.

I don't know if it is possible to embed the donation section from the donation website. In case it isn't, the 2nd option is better.

@NoamGaash What do you think?

NoamGaash commented 1 month ago

@iDanVas @VSciFlight I'm not sure I understand your suggestions. The JGive donation section can't be embedded (it's too expensive for us). That's why we have an image that links to the JGive donation page.

Right now we have two ways to make a donation:

We want to change the sidebar menu item so it will open a modal instead of opening the JGive link.

What do I miss here?

NoamGaash commented 1 month ago

One moment. This issue is not even about the sidebar link. We have two buttons. just delete one of them

NoamGaash commented 1 month ago

first place: https://github.com/hasadna/open-bus-map-search/blob/main/src/layout/header/Header.tsx#L23-L25 second place: https://github.com/hasadna/open-bus-map-search/blob/main/src/routes/index.tsx#L104-L123

supervxn commented 2 weeks ago

@iDanVas נעים להכיר :-) שמחה לעבוד איתך בפרויקט