mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 04:56:03 +03:00
[GH-ISSUE #325] UX suggestions #282
Labels
No labels
2fa
I18N
PGP
Security
Security
account
advanced_search
advanced_search
announcement
api_login
authentication
awaiting feedback
blocker
bug
bug
bug
calendar
config
contacts
core
core
devops
docker
docs
duplicate
dynamic_login
enhancement
epic
feature
feeds
framework
github
github
gmail_contacts
good first issue
help wanted
history
history
imap
imap_folders
inline_message
installation
keyboard_shortcuts
keyboard_shortcuts
ldap_contacts
mobile
need-ssh-access
new module set
nux
pop3
profiles
pull-request
question
refactor
release
research
saved_searches
smtp
strategic
tags
tests
themes
website
wordpress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/cypht#282
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Razvan0925 on GitHub (Mar 11, 2019).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/325
Originally assigned to: @jasonmunro on GitHub.
🗣 Suggestion
I have a few UX suggestions as I use this app everyday:
Keep up the good work :)
@jasonmunro commented on GitHub (Mar 12, 2019):
I'm not sure I understand the use case here. Why do you want them to remain checked?
The everything folder counter only appears in the page title after loading is complete, seems to be working correctly here. The unread folder has 2 counters, one in the title that will update after a refresh, and one in the folder list that should update about once a minute regardless of what page you are on. This is also working properly for me. Can you explain a bit more what broken behavior you are seeing?
We support these, but looks like I never added the module set to the sample ini file! add this to your hm3.ini:
modules[]=desktop_notificationsand rerun your config_gen.php script and they should be enabled. I will get that added to the sample ini file. Thanks for the feedback!
@Razvan0925 commented on GitHub (Mar 13, 2019):
That's the way it works in most email clients (gmail, yahoo, outlook). For me it is usefull because I receive many mails with notifications from servers that I manage, and once a day I select those mails, mark as read and delete them.
Another small thing is that in other email clients the selection toggle button, even if you have a few items selected, toggles between all and none, it does not inverse your selection :)
Just rechecked, you are right, it is working as you detailed, nothing broken :)
I have tried it but it is not working for me. I get an error in the console:
If it is working for you, maybe it's on my side. I will try to fix it and I will share what I've found.
@dumblob commented on GitHub (Mar 13, 2019):
From my point of view would be a nonsense to change the current behavior or even bloat the UI by adding an additional button.
@jasonmunro commented on GitHub (Mar 13, 2019):
I have to agree I also like the inverse behavior better than a standard toggle all :) However I have no issue with "leave checked items checked" after a message operation. @Razvan0925 insofar as the push error, pretty sure that is a browser setting situation. We outsource handling that to the push.js include, but it should prompt you the first time it wants to push a notice and it needs to be accepted or it ends up on a browser black-list (in chromium at least).
@Razvan0925 commented on GitHub (Mar 14, 2019):
The "leave checked items checked" would be nice for me, for the selection toggle I agree with you.
The push notifications is working now, it seems the browser refuses to push notification on non-secure or insecure connections, it should have been obvious 😁
@jasonmunro commented on GitHub (Mar 14, 2019):
checkboxes are no longer reset after a message action
github.com/jasonmunro/cypht@d23605b961@Razvan0925 commented on GitHub (Mar 21, 2019):
Another suggestion: I don't like the current notifications as they are covering the topbar buttons. Toast notifications would be a lot nicer.
@jasonmunro commented on GitHub (Mar 21, 2019):
@Razvan0925 how about something like this:

@Razvan0925 commented on GitHub (Mar 21, 2019):
Yes, it is nice 😃 I think it will look nice on desktop too
@jasonmunro commented on GitHub (Mar 22, 2019):
changes pushed to master for user message formatting
@jasonmunro commented on GitHub (Mar 22, 2019):
@Razvan0925 I think your original concerns have all been addressed, so I'm going to close this issue. I like to keep issues as specific and focused as possible (it just helps me stay organized). Fee free to open new issues for other UI suggestions, I appreciate the feedback!