Bug 135041 - radio buttons look like checkboxes in gtk3
Summary: radio buttons look like checkboxes in gtk3
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.3.0.0.alpha0+
Hardware: All Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2020-07-22 10:51 UTC by Justin L
Modified: 2023-07-11 04:42 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
TheseAreRadioButtons.jpg: how it looks on my screens. (246.09 KB, image/jpeg)
2020-07-22 10:55 UTC, Justin L
Details
elementary icon theme with kf5 backend (13.56 KB, image/png)
2020-07-22 13:25 UTC, Rizal Muttaqin
Details
7.5.5.1 gtk3 render of radios/checkbox (14.11 KB, image/png)
2023-07-10 18:37 UTC, Jojo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Justin L 2020-07-22 10:51:55 UTC
Radio buttons normally look like a dot in a circle.
Checkboxes normally look like a checkmark in a square.

However, now in 7.1 (and probably backported to 6.4.6) the radio button has a checkmark in it, and in the gtk3 theme the checked result looks like a square. See attachment 100290 [details] calc radio controls - which look like checkboxes.  (The gen theme also uses a checkmark which I don't really like, but at least it is clearly a circle.)

Since 7.1 commit dcc031d0cd3219dd77b66e1221fe966139829c1a
Author: Rizal Muttaqin on Tue Jun 9 14:33:26 2020 +0700
    Icon theme: tdf#133582 missing checkbox and radio button in gen env
Comment 1 Justin L 2020-07-22 10:55:35 UTC
Created attachment 163401 [details]
TheseAreRadioButtons.jpg: how it looks on my screens.
Comment 2 Rizal Muttaqin 2020-07-22 11:32:04 UTC
Yes, I confirmed this. I just follow elementary Human Interface Guideline here:

https://elementary.io/docs/human-interface-guidelines#radio-buttons

The point is I would like to follow upstream as close as possible. I would like to ask one of main elementary OS developer regarding this issue which already join LibreOffice Telegram Design group.
Comment 3 Justin L 2020-07-22 12:48:21 UTC
OK. I think in the particular situation / document I provided, the background of the button must be coming from somewhere other than the cell - so that the transparent-edges (of the square) are showing a black color from somewhere instead of the white background of the cell - making it look worse than it ought to look.

So there might be an implementation problem with controls that is more at fault here.
Comment 4 Xisco Faulí 2020-07-22 13:24:19 UTC
Regression introduced by:

https://cgit.freedesktop.org/libreoffice/core/commit/?id=86ea64f216819696cd86d1926aff0a138ace2baf

author	Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk>	2019-02-15 13:14:32 +0100
committer	Tomaž Vajngerl <quikee@gmail.com>	2019-04-03 11:57:08 +0200
commit 86ea64f216819696cd86d1926aff0a138ace2baf (patch)
tree db513803abc9dc255d27c0f08cba6d6d0c9ef1d9
parent 994b41a6c69d20637dcb95894c385f5c0102d600 (diff)
Support for native 32bit Bitmap in VCL and SVP (cairo) backend

Bisected with: bibisect-linux64-6.3

Adding Cc: to Tomaž Vajngerl
Comment 5 Rizal Muttaqin 2020-07-22 13:25:42 UTC
Created attachment 163406 [details]
elementary icon theme with kf5 backend

(In reply to Justin L from comment #3)
> OK. I think in the particular situation / document I provided, the
> background of the button must be coming from somewhere other than the cell -
> so that the transparent-edges (of the square) are showing a black color from
> somewhere instead of the white background of the cell - making it look worse
> than it ought to look.
> 
> So there might be an implementation problem with controls that is more at
> fault here.

I confirm the black edges in gtk3 backend. But with kf5 it looks OK (see the attached screenshot)

Version: 7.1.0.0.alpha0+
Build ID: 9006cbf6a13317a386194d6857f22391464c2aa0
CPU threads: 4; OS: Linux 4.20; UI render: default; VCL: kf5
Locale: id-ID (id_ID.UTF-8); UI: en-US
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2020-07-16_00:51:18
Calc: threaded
Comment 6 Rizal Muttaqin 2020-07-22 13:27:06 UTC Comment hidden (off-topic)
Comment 7 Rizal Muttaqin 2020-07-24 16:42:30 UTC
(In reply to Rizal Muttaqin from comment #2)

> The point is I would like to follow upstream as close as possible. I would
> like to ask one of main elementary OS developer regarding this issue which
> already join LibreOffice Telegram Design group.

Here is the answer from him:

https://blog.elementary.io/why-we-use-checks-in-checkboxes-and-radio-buttons/
Comment 8 Justin L 2020-07-28 08:11:01 UTC
I find it very telling that they even had to write an article a couple of years ago about why their bad idea wasn't so bad.

This phrase is key. 
> "Checkboxes and radio buttons are honestly used pretty sparingly 
> in elementary OS, 
> and even more sparingly together. When they are used, it’s fairly obvious
> from context whether it’s a multi-select or exclusive-select situation."

As attachment 100290 [details] shows, we don't have control over whether the use of radio buttons or checkboxes will be obvious from the context - since these are user-created items. Nor could we claim that they are sparingly used.

And I strongly disagree this this is just "historical differences for those subtly different behaviours". The distinction is not subtle at all.
Comment 9 QA Administrators 2022-07-29 04:36:19 UTC Comment hidden (obsolete, spam)
Comment 10 Jojo 2023-07-10 18:37:08 UTC
Created attachment 188298 [details]
7.5.5.1 gtk3 render of radios/checkbox

Setting the checkmarks in radio buttons issue aside..  The latest 7.5.5.1 gtk3 renders without black edges.

Version: 7.5.5.1 (X86_64) / LibreOffice Community
Build ID: 2c5e46c1980ec5241359fd65d751dc518205e7af
CPU threads: 4; OS: Linux 6.3; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 11 Justin L 2023-07-10 23:02:36 UTC
(In reply to Jojo from comment #10)
>  The latest 7.5.5.1 gtk3 renders without black edges.
Fixed in LO 7.2 with commit cd09fc9451897e6efedbf9f5e1d5b9bd96e65cb5
Author: Luboš Luňák on Mon Mar 22 19:06:15 2021 +0100
    do not enable mbSupportsBitmap32 for headless (tdf#141171)
    
    As said e.g. in 994b8e52fc02c7468a24 and 84f84f59ce7c83a99e4e340071d,
    LO code is not yet fully ready for 32bit bitmaps and e.g. PDF export
    code mishandles it.

Obviously this is not going to be a battle that I win, so with the radio button now at least looking like a circle, I'll mark it as fixed.
Comment 12 Rizal Muttaqin 2023-07-11 04:42:07 UTC
Personally, I prefer regular radio button rather a radio button with checkmark, some goes with save button in elementary theme.