Bug 140838

Summary: elementary: New Arrow Style for Undo and Redo
Product: LibreOffice Reporter: Rizal Muttaqin <rizmut>
Component: UIAssignee: Rizal Muttaqin <rizmut>
Status: VERIFIED FIXED    
Severity: enhancement CC: 79045_79045, xiscofauli
Priority: medium    
Version: 7.2.0.0.alpha0+   
Hardware: All   
OS: All   
Whiteboard: target:7.2.0 target:7.1.2
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 120949    
Attachments: New elementary Undo & Redo icons

Description Rizal Muttaqin 2021-03-06 09:20:35 UTC
See the commit from elementary icon upstream:

https://github.com/elementary/icons/commi/54dcc3648acd1e14877d8ff6a51bbfde3b660d6b
Comment 1 Roman Kuznetsov 2021-03-06 09:37:12 UTC
Page not found =((In reply to Rizal Muttaqin from comment #0)
> See the commit from elementary icon upstream:
> 
> https://github.com/elementary/icons/commi/
> 54dcc3648acd1e14877d8ff6a51bbfde3b660d6b

Page not found
Comment 2 Roman Kuznetsov 2021-03-06 09:38:56 UTC
(In reply to Roman Kuznetsov from comment #1)
> Page not found =((In reply to Rizal Muttaqin from comment #0)
> > See the commit from elementary icon upstream:
> > 
> > https://github.com/elementary/icons/commi/
> > 54dcc3648acd1e14877d8ff6a51bbfde3b660d6b
> 
> Page not found

because there is a mistake in the address, should be

https://github.com/elementary/icons/commit/54dcc3648acd1e14877d8ff6a51bbfde3b660d6b

And Rizal, please add screenshots for comparize before vs. after. Thanks
Comment 3 Rizal Muttaqin 2021-03-06 09:48:13 UTC
Created attachment 170266 [details]
New elementary Undo & Redo icons

(In reply to Roman Kuznetsov from comment #2)

> because there is a mistake in the address, should be
> 
> https://github.com/elementary/icons/commit/
> 54dcc3648acd1e14877d8ff6a51bbfde3b660d6b
> 

Thanks, sorry for wrong link

> And Rizal, please add screenshots for comparize before vs. after. Thanks

See the attachment
Comment 4 Roman Kuznetsov 2021-03-06 10:00:32 UTC
I don't see a many difference, but I don't have objections too

ps: I think there are no shadows for new Undo icons for 16 and 24 px, but there is a shadow for 32px Undo icon.
Comment 5 Rizal Muttaqin 2021-03-06 10:14:23 UTC
(In reply to Roman Kuznetsov from comment #4)
> I don't see a many difference, but I don't have objections too

Yes, elementary folks don't want a radical change, see further explanation here: https://blog.elementary.io/look-and-feel-changes-elementary-os-6/

> ps: I think there are no shadows for new Undo icons for 16 and 24 px, but
> there is a shadow for 32px Undo icon.

The whole icons have shadow, you just need to see actual file. I won't change anything from upstream
Comment 6 Commit Notification 2021-03-06 16:05:19 UTC
Rizal Muttaqin committed a patch related to this issue.
It has been pushed to "master":

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

elementary: tdf#140838 new Undo & Redo icons

It will be available in 7.2.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 7 Commit Notification 2021-03-06 16:06:30 UTC
Rizal Muttaqin committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/9eeb580f5c57918484baff74ca052a96ed1056b3

elementary: related to tdf#140838 new Undo & Redo icons

It will be available in 7.2.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 8 Commit Notification 2021-03-07 07:40:21 UTC
Rizal Muttaqin committed a patch related to this issue.
It has been pushed to "libreoffice-7-1":

https://git.libreoffice.org/core/commit/982c579fbe59812900f5d181eea0c9b7efa78a8a

elementary: related to tdf#140838 new Undo & Redo icons

It will be available in 7.1.2.

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 9 Commit Notification 2021-03-07 07:41:32 UTC
Rizal Muttaqin committed a patch related to this issue.
It has been pushed to "libreoffice-7-1":

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

elementary: tdf#140838 new Undo & Redo icons

It will be available in 7.1.2.

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 10 Rizal Muttaqin 2021-03-30 14:33:59 UTC
fixed in

Version: 7.2.0.0.alpha0+ / LibreOffice Community
Build ID: a22357213807fa46b903e7d040809cff194c39af
CPU threads: 4; OS: Linux 5.4; 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: 2021-03-29_12:01:23
Calc: threaded
Comment 11 Stéphane Guillou (stragu) 2021-11-30 13:19:06 UTC
verified as fixed in:

Version: 7.2.0.2 / LibreOffice Community
Build ID: 614be4f5c67816389257027dc5e56c801a547089
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded