Bug 144637 - Breeze theme: Inconsistent icons for the various cell border types
Summary: Breeze theme: Inconsistent icons for the various cell border types
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
7.2.1.1 rc
Hardware: All Windows (All)
: medium minor
Assignee: Rizal Muttaqin
URL:
Whiteboard: target:7.3.0
Keywords:
Depends on:
Blocks: Icon-Theme-Breeze
  Show dependency treegraph
 
Reported: 2021-09-21 10:14 UTC by Ming Hua
Modified: 2021-11-20 11:57 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot of the dropdown from "Cell Border" toolbar button (2.70 KB, image/png)
2021-09-21 10:14 UTC, Ming Hua
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ming Hua 2021-09-21 10:14:12 UTC
Created attachment 175162 [details]
Screenshot of the dropdown from "Cell Border" toolbar button

Description:
The dropdown from the "Cell Borders" toolbar button in Calc contain many icons representing different ways/schemes to add cell borders.  However while using Breeze icon style, I noticed one of these icons, the "all outer and inner borders" one, has a drastic different style than others.  See attached screenshot.

I tested other icon styles, and Breeze seems to be the only one having this problem.

Steps to Reproduce:
1. Start Calc, the default standard interface should have the "Formatting" toolbar visible, on it there should be "Borders" button that sets cell borders.
2. Switch to "Breeze" icon style via Tools > Options > LibreOffice > View > Icon Style.
2. Click on the "Borders" button, there should be a dropdown with many icons representing different ways to set cell borders.  Observe that one of them doesn't look like others.

Version Information:
First encountered with 7.2.1 on Windows 10:
Version: 7.2.1.2 (x64) / LibreOffice Community
Build ID: 87b77fad49947c1441b67c559c339af8f3517e22
CPU threads: 2; OS: Windows 10.0 Build 19043; UI render: Skia/Raster; VCL: win
Locale: zh-CN (zh_CN); UI: zh-CN
Calc: threaded

Also reproducible with 7.3/master daily build:
Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: a9cc066a86c6bd3423c5802c5a4eded55a50c754
CPU threads: 2; OS: Windows 10.0 Build 19043; UI render: default; VCL: win
Locale: zh-CN (zh_CN); UI: zh-CN
Calc: threaded
Comment 1 Rizal Muttaqin 2021-09-22 03:52:22 UTC
I wanted to report this but forgot, confirming also in:

Version: 7.0.6.2
Build ID: 00(Build:2)
CPU threads: 4; OS: Linux 5.11; UI render: default; VCL: kf5
Locale: id-ID (id_ID.UTF-8); UI: id-ID
Ubuntu package version: 1:7.0.6-0ubuntu0.20.04.1~lo1
Calc: threaded
Comment 2 Commit Notification 2021-10-04 08:25:56 UTC
Rizal Muttaqin committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/af291666b3316aba2b35d3b8358f4cd42c553587

Breeze: tdf#144637 update & fix table cell border

It will be available in 7.3.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 3 Ming Hua 2021-11-20 11:57:41 UTC
Confirmed with 7.3/master daily build on Windows 10:
Version: 7.3.0.0.alpha1+ (x64) / LibreOffice Community
Build ID: 4be0ae19065b1b50870bc0b2a28189ad39c96a8a
CPU threads: 2; OS: Windows 10.0 Build 19043; UI render: default; VCL: win
Locale: zh-CN (zh_CN); UI: zh-CN
Calc: threaded

Thanks rizmut for fixing this issue!