Bug 163897 - Removing conditional formats behaviour is not consistent
Summary: Removing conditional formats behaviour is not consistent
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
25.2.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:25.8.0
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Conditional-Formatting-Editing
  Show dependency treegraph
 
Reported: 2024-11-14 14:14 UTC by Xisco Faulí
Modified: 2025-02-19 16:16 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Xisco Faulí 2024-11-14 14:14:00 UTC
Steps to reproduce:
1. Open attachment 196417 [details] from bug 162938
2. Format - Condition - Manage
3. Click on Remove until all conditional formattings are removed
4. Click on Add
5. Cancel the new dialog

-> All the conditional formats are back

Version: 25.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 7a7ba9cbee91485a9254949d1594352b3629c070
CPU threads: 8; OS: Linux 6.1; UI render: default; VCL: gtk3
Locale: es-ES (es_ES.UTF-8); UI: en-US
Calc: threaded
Comment 1 Xisco Faulí 2024-11-14 14:17:01 UTC
Regression introduced by:

commit 20350c7f1085b0a73191e318e14f2dbd69553e0b	[log]
author	Pranam Lashkari <lpranam@collabora.com>	Wed Jun 26 00:07:23 2024 +0100
committer	Caolán McNamara <caolan.mcnamara@collabora.com>	Thu Aug 15 12:28:10 2024 +0200
tree 56424d3e35758dc039003648ec76bf964ec23598
parent f8da93d33ddadb6f60c385b5ee93a55d303d916f [diff]

sc: tdf#162475 manage conditional format with simpler dialog

Bisected with: bibisect-linux64-25.2
Comment 2 Xisco Faulí 2025-02-19 11:09:24 UTC
The commit mentioned in comment 1 was reverted in

commit f643a98b2b60ccad63f7b4647ff6bc0141df0718
Author: Pranam Lashkari <lpranam@collabora.com>
Date:   Mon Nov 25 19:47:26 2024 +0530

    Revert "sc: manage conditional format with simpler dialog"

Closing as RESOLVED FIXED
Comment 3 Commit Notification 2025-02-19 16:16:11 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

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

tdf#163897: sc: Add UItest

It will be available in 25.8.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.