novelWriter icon indicating copy to clipboard operation
novelWriter copied to clipboard

Manuscript Build Settings - Close not asking for confirmation after changes (2.5 beta 1)

Open awqk opened this issue 1 year ago • 7 comments

  1. open Manuscript Build Settings

  2. change something on 1st page

  3. close Manuscript Build Settings

  4. question: save yes/no

  5. open Manuscript Build Settings

  6. go to any other page and change something

  7. close Manuscript Build Settings

  8. changes are silently discarded

It's also possible to get the settings in an inconsistent state, though I am unable to reproduce this. I had a situation where the scene headers disappeared from the preview unless the alt scene headers were hidden.

awqk avatar Jun 13 '24 14:06 awqk

The filters are applied as-is. This is not a bug, but I agree it is not intuitive. The "Apply" only actually applies to the settings themselves.

If you make setting changes, and don't Apply or Save, and then close the whole Manuscript tool, the setting panels are auto-cancelled, that's correct. Also not a bug, but maybe again not intuitive.

Finally, the reason some settings are disappearing is because you are probably also opening the same project with another version of the app. This converts your build setting to an older format when you open them. If you opened the same project with, say, 2.4 RC 1 and 2.3 back in that release cycle, you would lose the Alt Scene Header setting for instance.

vkbo avatar Jun 14 '24 05:06 vkbo

The filters are applied as-is. This is not a bug, but I agree it is not intuitive. The "Apply" only actually applies to the settings themselves. If you make setting changes, and don't Apply or Save, and then close the whole Manuscript tool, the setting panels are auto-cancelled, that's correct. Also not a bug, but maybe again not intuitive.

So the first page (Selection) is not a setting, and therefore it isn't auto-cancelled, whereas the other pages are settings? I agree this is not intuitive :flushed:.

Finally, the reason some settings are disappearing is because you are probably also opening the same project with another version of the app. This converts your build setting to an older format when you open them. If you opened the same project with, say, 2.4 RC 1 and 2.3 back in that release cycle, you would lose the Alt Scene Header setting for instance.

Didn't. Just installed 2.5b1 over 2.4.3 and never returned. Perhaps my description was misleading: it wasn't the settings disappearing, it was the scene headers in the preview text that didn't show up unless the alt scene headers were hidden, as if there was some confusion whether the scene headers in the documents were alt scene headers or not. There were several sessions with 2.5b1 before I noticed this. I tried a lot of hiding and showing with a unexpected results. Problem only vanished after close+open.

awqk avatar Jun 14 '24 09:06 awqk

So the first page (Selection) is not a setting, and therefore it isn't auto-cancelled, whereas the other pages are settings? I agree this is not intuitive 😳.

No, it's not a setting per-say. It's meant to be a dynamic filter mechanism to be used live while you build so you don't have to hit "Apply". I'm not sure how to distinguish this feature in the dialog, but I do want to redesign the build settings dialog a little, which may help make this distinction as a side effect.

Didn't. Just installed 2.5b1 over 2.4.3 and never returned. Perhaps my description was misleading: it wasn't the settings disappearing, it was the scene headers in the preview text that didn't show up unless the alt scene headers were hidden, as if there was some confusion whether the scene headers in the documents were alt scene headers or not. There were several sessions with 2.5b1 before I noticed this. I tried a lot of hiding and showing with a unexpected results. Problem only vanished after close+open.

Ah, ok. There could be a bug there. I would need a point by point list to reproduce, as it isn't clear to me how this is triggered.

I plan to release 2.5 RC 1 very soon, so it would be good to get in any fixes before that for a better release candidate before the final release.

vkbo avatar Jun 14 '24 09:06 vkbo

Ah, ok. There could be a bug there. I would need a point by point list to reproduce, as it isn't clear to me how this is triggered.

That's were I am confused myself, it could have involved closing the build tool without closing the build settings first. Or it was a pilot error, me not hitting Apply before updating the preview, repeatedly (as you said, it's not intuitivie), though I'm sure I would have tried the Save button at some point, it's hard to overlook.

awqk avatar Jun 14 '24 09:06 awqk

Using Apply before updating preview would be pretty standard behaviour I would expect. There is no way around that unless I make all settings "live".

I can make the filter section work on a copy of the build settings and make them also activate on Apply/Save. At least then it would be more consistent, but you would have to hit apply before each build it you're, say, building multiple manuscripts in a multi-novel project, one for each novel. That's precisely why the filter is applied directly.

vkbo avatar Jun 14 '24 10:06 vkbo

Sounds good. In a multi-novel project, I wouldn't have thought of using different filter settings on the fly to create the manuscripts. Instead, creating a build setting for each individual novel would have been my first idea.

awqk avatar Jun 14 '24 10:06 awqk

Well, the feature was built to satisfy a feature request that specifically related to building for individual novels, or even chapters or single documents. So the filtering was treated as a separate feature. In the original build settings layout they were even on a panel on their own next to the settings, instead of on a tab. However, that used up too much screen space, so I redesigned it.

I'm a little reluctant to abandon this use case, as I also use this myself, so let's see how it looks after I've restructured the settings dialog. Of course, pressing Apply isn't a huge disadvantage, so it doesn't prevent this use case.

vkbo avatar Jun 14 '24 10:06 vkbo

As of 2.7, the Build Settings dialog works on a copy of the current build settings, so changes are no longer applied directly and not until Apply or OK is clicked. See #2351.

I believe can be closed as completed.

vkbo avatar Jun 01 '25 13:06 vkbo

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs or feature requests.

github-actions[bot] avatar Oct 01 '25 03:10 github-actions[bot]