mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-05 16:49:47 +03:00
Table editing icons hardly decipherable #2763
Closed
opened 2026-02-05 05:06:03 +03:00 by OVERLORD
·
12 comments
No Branch/Tag Specified
development
l10n_development
further_theme_development
release
llm_only
vectors
v25-11
docker_env
drawio_rendering
user_permissions
ldap_host_failover
svg_image
prosemirror
captcha_example
fix/video-export
v25.12.3
v25.12.2
v25.12.1
v25.12
v25.11.6
v25.11.5
v25.11.4
v24.11.4
v25.11.3
v25.11.2
v25.11.1
v25.11
v25.07.3
v25.07.2
v25.07.1
v25.07
v25.05.2
v25.05.1
v25.05
v25.02.5
v25.02.4
v25.02.3
v25.02.2
v25.02.1
v25.02
v24.12.1
v24.12
v24.10.3
v24.10.2
v24.10.1
v24.10
v24.05.4
v24.05.3
v24.05.2
v24.05.1
v24.05
v24.02.3
v24.02.2
v24.02.1
v24.02
v23.12.3
v23.12.2
v23.12.1
v23.12
v23.10.4
v23.10.3
v23.10.2
v23.10.1
v23.10
v23.08.3
v23.08.2
v23.08.1
v23.08
v23.06.2
v23.06.1
v23.06
v23.05.2
v23.05.1
v23.05
v23.02.3
v23.02.2
v23.02.1
v23.02
v23.01.1
v23.01
v22.11.1
v22.11
v22.10.2
v22.10.1
v22.10
v22.09.1
v22.09
v22.07.3
v22.07.2
v22.07.1
v22.07
v22.06.2
v22.06.1
v22.06
v22.04.2
v22.04.1
v22.04
v22.03.1
v22.03
v22.02.3
v22.02.2
v22.02.1
v22.02
v21.12.5
v21.12.4
v21.12.3
v21.12.2
v21.12.1
v21.12
v21.11.3
v21.11.2
v21.11.1
v21.11
v21.10.3
v21.10.2
v21.10.1
v21.10
v21.08.6
v21.08.5
v21.08.4
v21.08.3
v21.08.2
v21.08.1
v21.08
v21.05.4
v21.05.3
v21.05.2
v21.05.1
v21.05
v21.04.6
v21.04.5
v21.04.4
v21.04.3
v21.04.2
v21.04.1
v21.04
v0.31.8
v0.31.7
v0.31.6
v0.31.5
v0.31.4
v0.31.3
v0.31.2
v0.31.1
v0.31.0
v0.30.7
v0.30.6
v0.30.5
v0.30.4
v0.30.3
v0.30.2
v0.30.1
v0.30.0
v0.29.3
v0.29.2
v0.29.1
v0.29.0
v0.28.3
v0.28.2
v0.28.1
v0.28.0
v0.27.5
v0.27.4
v0.27.3
v0.27.2
v0.27.1
v0.27
v0.26.4
v0.26.3
v0.26.2
v0.26.1
v0.26.0
v0.25.5
v0.25.4
v0.25.3
v0.25.2
v0.25.1
v0.25.0
v0.24.3
v0.24.2
v0.24.1
v0.24.0
v0.23.2
v0.23.1
v0.23.0
v0.22.0
v0.21.0
v0.20.3
v0.20.2
v0.20.1
v0.20.0
v0.19.0
v0.18.5
v0.18.4
v0.18.3
v0.18.2
v0.18.1
v0.18.0
v0.17.4
v0.17.3
v0.17.2
v0.17.1
v0.17.0
v0.16.3
v0.16.2
v0.16.1
v0.16.0
v0.15.3
v0.15.2
v0.15.1
v0.15.0
v0.14.3
v0.14.2
v0.14.1
v0.14.0
v0.13.1
v0.13.0
v0.12.2
v0.12.1
v0.12.0
v0.11.2
v0.11.1
v0.11.0
v0.10.0
v0.9.3
v0.9.2
v0.9.1
v0.9.0
v0.8.2
v0.8.1
v0.8.0
v0.7.6
v0.7.5
v0.7.4
v0.7.3
0.7.2
v.0.7.1
v0.7.0
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.0
Labels
Clear labels
🎨 Design
📖 Docs Update
🐛 Bug
🐛 Bug
:cat2:🐈 Possible duplicate
💿 Database
☕ Open to discussion
💻 Front-End
🐕 Support
🚪 Authentication
🌍 Translations
🔌 API Task
🏭 Back-End
⛲ Upstream
🔨 Feature Request
🛠️ Enhancement
🛠️ Enhancement
🛠️ Enhancement
❤️ Happy feedback
🔒 Security
🔍 Pending Validation
💆 UX
📝 WYSIWYG Editor
🌔 Out of scope
🔩 API Request
:octocat: Admin/Meta
🖌️ View Customization
❓ Question
🚀 Priority
🛡️ Blocked
🚚 Export System
♿ A11y
🔧 Maintenance
> Markdown Editor
pull-request
Mirrored from GitHub Pull Request
No Label
🔨 Feature Request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/BookStack#2763
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
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 @Wookbert on GitHub (Apr 26, 2022).
Describe the feature you'd like
More a design flaw, than a bug. But also not quite a feature request.
Compared to the previous design I find the new icons for table editing hardly decipherable, especially at very dense display resolutions (as smaller the icons, as more difficult to distinguish). If not replacing those icons entirely, one should at least make the X in the icons red and the + green.
Describe the benefits this would bring to existing BookStack users
Better usability.
Can the goal of this request already be achieved via other means?
No.
Have you searched for an existing open/closed issue?
How long have you been using BookStack?
1 to 5 years
Additional context
No response
@ssddanbrown commented on GitHub (Apr 28, 2022):
Thanks for the input @Wookbert
Previous Icon Designs for Reference
TinyMCE 6 icons appear to be the same as the current, newer icons.
Personally, I don't see too much of a difference between these, and think the newer icons actually make a little more sense in their representation although differentiating between the cross and plus is maybe slightly harder in the new.
Is there anything particular in the new icons that you find particularly troublesome in comparison to the old ones or is it mainly due to distinguishing the cross/plus?
@Wookbert commented on GitHub (Apr 28, 2022):
No idea what you display size and resolution is, but I use a 14" MacBook Pro with 3024 × 1964 px. resolution, which is set to "more space" under scaling (looks as 1800 x 1169). As smaller the icons on the screen are (due to high resolution and scaling), as harder it becomes to identify the individual icons and their function. You might get a better idea if you view the icon images in this thread on a small Smartphone screen or decrease Zoom below 100% on your Desktop browser.
Keep in mind that you personally might be able to press the respective buttons blindly, but anyone who’s using the function only occasionally, has to identify them actually by their graphical meaning.
As for previous TinyMCE icons: These are clearly much easier/better to identify as the X and + are both bigger, bolder and the + in particular has more white space. Every designer knows that: As smaller a line graphic gets, as more space between the lines are needed. That’s why same logos can't be printed on ball pens (if they still should be recognizable).
@ssddanbrown commented on GitHub (May 12, 2022):
Sure, as I alluded to in my previous comment, I think the older icons are graphically clearer but the new icons are a better representation of their action.
I don't use these actions too much, I have often deferred to the hover text, both before and after the icon change.
I looked to google material icons for some alternatives but they don't have many options for these cases.
Getting color involved is a little messy and adds some complication but doable. Coloring small details can often appear awkward, and we may have to have different colors for light/dark modes. Would need to test things out.
@Wookbert commented on GitHub (May 12, 2022):
IMO there is ZERO room for misinterpreting the previous icons, but a SERIOUS problem deciphering the current ones at small representation and /or bad eye sight. Can’t you simply roll back to the previous graphics, or does that involve tweaking TinyMCE now and in future releases?
@ssddanbrown commented on GitHub (May 12, 2022):
We could probably find the old icon sources then override the newer icons, assuming the old ones were also SVGs, but then they won't really fit with the theme of the others too well. Personally I still have a slight preference for the new icons.
@Wookbert commented on GitHub (May 12, 2022):
I can lend you my nearly 50 year old eyes, plus my 14" 2021 MacBook and you'll instantly change your mind. 🤪
(Say no more... you are going for the MacBook only).
@Wookbert commented on GitHub (May 12, 2022):
OK... Coincidentally creating a table in BookStack right now and noticed that in the menus, the disable icons in medium grey are a bit easier readable (sic!) than the high contrast black editing icons which appear under an existing table. But I understand that this color scheme is reserved for the ghosted items and would require the ghosted icons to be brighter.
Still, if we'd make the editing icons in the same grey, it would help already. And if we could turn the + green and the X red I think it would be perfect.
Forward me the SVG package and I'll create an alternative, improved version. Both color added to black/medium grey and medium grey/lighter grey.
@ssddanbrown commented on GitHub (Jul 24, 2022):
Here's an easy "Custom HTML Head Content" addition which should work on a current instance:
This will selectively change the color of the icons as a whole. Not tested against dark mode though.
I did play around with changing the +/x color but they don't look right, the tiny hint of color made the icons harder to parse from my perspective.
@ssddanbrown commented on GitHub (Sep 30, 2022):
Just spent a little time with the icons, seeing if I use the modern & monotone styling of the newer icons while removing some of the visual noise. In the image is the current icons and then the tweaked icons below.
I've tried to scale this screenshot (Within the display of this issue, not the file itself) to be representative of the matching web-viewed size as 100% zoom.
Do you find the second set to be an improvement?
Commit for my reference:
953402f2eb@Wookbert commented on GitHub (Oct 1, 2022):
Second design is definitely an improvement in my opinion. Any chance to additionally make the +sses green and the Xsses red? (Only the + and X, not the entire icons).
@ssddanbrown commented on GitHub (Oct 1, 2022):
I did play about with that back in July, but it only seemed to make the icons harder to read. The small amount of color seemed to not be instantly recognizable and caused more visual distraction.
@ssddanbrown commented on GitHub (Oct 2, 2022):
Have now merged the tweaked icons into the development branch, to be part of the next feature release.
Hopefully between these changes, the hover-labels that already exist and, if applied, the color tweak above, these icons should be significantly more decipherable so I will therefore close this off.