mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 13:05:53 +03:00
[GH-ISSUE #766] Menu for list controls on mobile has an incorrect icon #471
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#471
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 @jasonmunro on GitHub (Sep 13, 2023).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/766
Originally assigned to: @Yannick243 on GitHub.
🐛 Bugreport
The list controls for mobile defined in modules/core/message_list_functions.php at line 476 has the following:
However the icon that shows up is an "x"
Version & Environment
master branch on debian unstable
Steps to reproduce
@jasonmunro commented on GitHub (Sep 14, 2023):
Looks like the problem is the Hm_Image_sources "three_dot" svg definition is incorrect, located here:
https://github.com/cypht-org/cypht/blob/master/lib/output.php#L133
that value should be changed to something like:
Might need to tweak some CSS after fixing the icon definition to line it up properly.
@marclaporte commented on GitHub (Nov 1, 2023):
@jasonmunro Are you OK with this fix?
@Yannick243 commented on GitHub (Feb 6, 2024):
Pull request already merged: https://github.com/cypht-org/cypht/pull/768