[Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2024-04-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #14 from Commit Notification 
 ---
Caolán McNamara committed a patch related to this issue.
It has been pushed to "libreoffice-24-2":

https://git.libreoffice.org/core/commit/843b3abd2ab05c66533b12ef6a4b155247f99e58

Resolves: tdf#154715 don't use last-used field setting when editing field

It will be available in 24.2.3.

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.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2024-04-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:24.8.0   |target:24.8.0 target:24.2.3

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2024-04-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Caolán McNamara  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #13 from Caolán McNamara  ---
seems to do something more sensible now

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2024-04-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #12 from Commit Notification 
 ---
Caolán McNamara committed a patch related to this issue.
It has been pushed to "master":

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

Resolves: tdf#154715 don't use last-used field setting when editing field

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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2024-04-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:24.8.0

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2024-04-07 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Caolán McNamara  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #11 from Caolán McNamara  ---
I think what the bisect identified is the commit that restored the ability of
this dialog to read the previous saved settings used in that page.

What I think is then funky is that the page has "IsFieldEdit" which
distinguishes between the page used to edit a field, vs the other case when its
for inserting a field.

But even if "IsFieldEdit" is set, then it uses the saved settings to restore
the last used settings. What is particularly off is

https://cgit.freedesktop.org/libreoffice/core/tree/sw/source/ui/fldui/fldref.cxx#n270

where we have: if( !IsRefresh() ) used twice, one inside the other, but if was
true in the outer use, it would be true in the inner (all the way back to the
initial commit). And IsRefresh is only true I think for the case of using the
"next/back" button to go to edit another field.

Which suggests that if you click next and then back that you then get the right
results

I think one of those IsRefresh was intended to be IsFieldEdit

https://gerrit.libreoffice.org/c/core/+/165876

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-05-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #10 from sdc.bla...@youmail.dk ---
Additional information

1.  Use the attached test file (attachment 187437)

2.  place cursor at end of document, open Insert Field dialog (Ctrl+F2) -
DocInformation tab.

3. Select "Type" "Revision number" and Insert.

4. Now edit any of the inserted cross-ref fields.

Actual and expected:  Field is opened with correct type, selection, refer
using.

5.  Insert a new cross-reference field, then edit any existing cross-reference
field.

Actual:  goes back to OP reported behavior (i.e., uses last selected
type/selection/refer using).


==>  inserting a docinformation field seems to "update" something that allows
appropriate editing of cross-reference fields, which gets lost again after
inserting a new cross-reference field.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-05-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #9 from sdc.bla...@youmail.dk ---
(In reply to Heiko Tietze from comment #8)
> (Was wondering if we talk about Insert + Document, Insert + CrossRef...
> expecting the last choice again.)
as noted in bug summary and in OP, the problem is in relation to fields entered
from the Cross-references tab in the Insert Field dialog.

afacit, there are no problems with editing fields that were inserted from the
Document or DocInformation tab (i.e., editing those fields opens to the correct
properties for the field).

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-05-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Heiko Tietze  changed:

   What|Removed |Added

   Keywords|needsUXEval |
 Status|NEEDINFO|NEW
 CC|libreoffice-ux-advise@lists |heiko.tietze@documentfounda
   |.freedesktop.org|tion.org
 Blocks||109265

--- Comment #8 from Heiko Tietze  ---
I insert Document-Sender and the this will be shown even for page number, for
instance. This should be fixed. Not only the type of field (resp. tab) and the
type should to be set according the actual field content but also the
selection/format.

(Was wondering if we talk about Insert + Document, Insert + CrossRef...
expecting the last choice again.)


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=109265
[Bug 109265] [META] Remember dialog's previous settings issues
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-05-22 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #7 from sdc.bla...@youmail.dk ---
Created attachment 187437
  --> https://bugs.documentfoundation.org/attachment.cgi?id=187437=edit
file to demonstrate problem

(In reply to Heiko Tietze from comment #6)
> please share a better STR for the actual problem.
Demonstration file with instructions is attached, but any document with
cross-reference fields can be used.

STR.
1. (to prepare test document, such as the attached, insert some
cross-references with a variety of "type", "selection" and "refering using")

2.  Insert a new cross-reference (of any kind).

3.  Edit one of the previous cross-reference fields.

Actual:  When a previous cross-reference field is edited, then the "Type",
"Selection", and "Refer using" corresponds to the last used choice.

Expected:  That "Type", "Selection", and "Refer using" corresponds to the
values of the field being edited.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-05-15 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Heiko Tietze  changed:

   What|Removed |Added

 Status|NEW |NEEDINFO

--- Comment #6 from Heiko Tietze  ---
Basically it's desirable to open a tabbed dialog at the right place. But I
wouldn't treat it as a huge mistake if that's not the fact in some situations
since all tabs should have some close relation with the target.

When it comes to fields it might be a bit more difficult. If the first field is
a cross-reference, followed by some docinformation, you wont be able to go from
one to the other per edit field. If the fields are in the same context
everything is updated as expected.

So yes in general, and please share a better STR for the actual problem.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-04-28 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

sdc.bla...@youmail.dk changed:

   What|Removed |Added

   Keywords||needsUXEval
 CC||libreoffice-ux-advise@lists
   ||.freedesktop.org

--- Comment #5 from sdc.bla...@youmail.dk ---
(In reply to Buovjaga from comment #4)
> Just a note: I was looking at the type to be specific.
Will ask UXEval in relation to general issue.  Also, maybe this is an EasyHack

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-04-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #4 from Buovjaga  ---
(In reply to Buovjaga from comment #1)
> Bibisected with Linux 43max to 333282478a956709e648427b412d90f90fc1437b
> Related: fdo#75307 use the unique auto help id to identify tabpages

Just a note: I was looking at the type to be specific.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-04-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

sdc.bla...@youmail.dk changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=92
   ||688

--- Comment #3 from sdc.bla...@youmail.dk ---
Adding mention of enhancement request - bug 92688 (to remember the last used
cross-reference type for "insert - field")  -- in case it involves the same
underlying code.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-04-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

--- Comment #2 from Buovjaga  ---
Created attachment 186617
  --> https://bugs.documentfoundation.org/attachment.cgi?id=186617=edit
Example file

Here is a silly test file for accelerated testing.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-04-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

Buovjaga  changed:

   What|Removed |Added

   Keywords||bibisected, bisected,
   ||regression
   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=75
   ||307
Version|7.2.7.2 release |4.3.0.4 release
 CC||caol...@redhat.com,
   ||ilmari.lauhakangas@libreoff
   ||ice.org
 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1
  Regression By||Caolán McNamara

--- Comment #1 from Buovjaga  ---
Bibisected with Linux 43max to 333282478a956709e648427b412d90f90fc1437b
Related: fdo#75307 use the unique auto help id to identify tabpages

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 154715] "Edit Fields" for cross-reference fields should open on the type, format, and selection of the inserted field

2023-04-08 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=154715

sdc.bla...@youmail.dk changed:

   What|Removed |Added

 Blocks||114039


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=114039
[Bug 114039] [META] Fields dialog bugs and enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.