-
Notifications
You must be signed in to change notification settings - Fork 3
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
Show Hijri date on hover of Gregorian date #47
Comments
Another option: https://www.npmjs.com/package/hijri-date Though I don't think it brings any special value over the native date handling + day.js |
Assalamualaikum. If you would take my suggestion, display only the Hijri date. Everyone has the Gregorian date on their devices. It makes more sense when we visit an Islamic website to see the Hijri date. As for the Hijri corrections with respect to location, I have had the same problem. The way I have thought to solve this is to follow the Makkah calendar, and have a list of corrections corresponding to coordinate ranges and apply the appropriate correction. What do you think about this? |
Wa alaykum as salam wa rahmatullah @Akram9 😊🌷 Thank you so much for your good suggestions How lovely it would be to make Hijri dates the default! For sure, people can get Gregorian dates everywhere else, and Muslims should love to use the Hijri calendar. But the reason why I think the Georgian date should be default on Layl: Because it's essential for the user to double check the date Layl is using for calculations. In some rare cases, their device can report the wrong date, which is what Layl detects, so if Layl displays the Hijri date by default they might miss the error. But if Layl shows the wrong Gregorian date, people can quickly realize the mistake and correct it. So for accuracy and to allow double checking easily, the Georgian date is a sad necessity I think. But we can do something to make the Hijri date more prominent. Maybe some kind of fade in, where after maybe 10 seconds, the Hijri date appears and Gregorian is switched to the tool tip. Let me know your thoughts! And right now I'm focused on finishing support for translations in Layl, so we have time to make the best plan inshaAllah 👌🏽. You can also kindly help test the latest version of Layl here! https://deploy-preview-41--layl-beta.netlify.app/ |
Okay, now I understand why you need to display the Gregorian dates. In that case, what about putting both the dates side by side. Maybe like - 23 Sha'ban (5 Apr - 6Apr) or the Hijri date on top of the Gregorian one like The reason I would suggest to keep the Hijri date clearly visible rather than hidden is because many people may not know that thay can access Hijri under the Gregorian date. What do you think? |
The latest version works good. But it appears dark on SalamWeb, Vivaldi and Firefox, while it is white in Chrome. I hope I am doing something wrong. Also, in the last paragraph in the Sufone website there is a grammatical error. |
Jazakumullahu khayran! 🌻
Jazakumullahu khayran and I hope you have a good rest :) |
Wa iyyaka! 1.In sha Allah. 2.Yes, indeed it is following the theme of the browser. My Chrome theme was light, so Layl was light last night. Setting dark theme makes Layl dark too. So it is working perfectly fine. 3.No problem. |
A progressive enhancement to encourage the Hijri calendar. Can't make it the default since people could be confused and might not be able to check properly
Need to figure out how to show this is a feature (possibly a a grey, dotted underline), and to make it work on mobile (on tap)
Similar to #40
Can use this browser native code:
This uses the Umm al-Qura system (I think—we need to confirm), which of course not everybody follows, nor is it applicable to all locations. But it's a good approximation in general, and so this Hijri date should be shown an indication this is an approximate date.
The text was updated successfully, but these errors were encountered: