Bug 4239 - Preferences: Text Options Header Display modal is not modal
Summary: Preferences: Text Options Header Display modal is not modal
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: UI (show other bugs)
Version: 3.17.5
Hardware: All All
: P3 minor
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2019-08-23 23:05 UTC by Alexander Harkness
Modified: 2019-09-19 12:51 UTC (History)
0 users

See Also:


Attachments
Patch to resolve issue (626 bytes, patch)
2019-08-23 23:05 UTC, Alexander Harkness
no flags Details | Diff

Description Alexander Harkness 2019-08-23 23:05:39 UTC
Created attachment 1999 [details]
Patch to resolve issue

On tiling WMs it does not show up as a modal but instead a maximised window.

Included patch resolves the issue by making behaviour of prefs_display_header consistent with prefs_summary_column.
Comment 1 Paul 2019-08-24 11:21:10 UTC
Which tiling WMs?
Comment 2 Alexander Harkness 2019-08-24 15:30:55 UTC
> Which tiling WMs?

I use i3, but I imagine that the behaviour will be the same in similar WMs as well.

In any case, the patch attached just makes the behaviour of the modal in question consistent with other preferences windows, which is correct as far as I can tell.
Comment 3 Andrej Kacian 2019-08-24 18:01:02 UTC
While I do not argue that having the dialogs behave in the same way is a good thing, I wonder if disabling resizeability is not a bit heavy-handed way to signal to the window manager that this is a dialog window.

Alexander, can you please try this instead?
  gtk_window_set_type_hint(GTK_WINDOW(window), GDK_WINDOW_TYPE_HINT_DIALOG);
Comment 4 Alexander Harkness 2019-08-24 19:05:52 UTC
> Alexander, can you please try this instead?
>   gtk_window_set_type_hint(GTK_WINDOW(window), GDK_WINDOW_TYPE_HINT_DIALOG);

I added that after
  gtk_window_set_resizable(GTK_WINDOW (window), TRUE);
and it behaves as it should (as a modal/dialog window above the other parts of the UI), and it can still be resized.

I agree that disabling the ability to resize the windows really is a bit heavy-handed - perhaps the others should have theirs re-enabled?
Comment 5 Andrej Kacian 2019-08-24 21:29:51 UTC
Yes, we will likely need to go over all of our dialog windows and assign these hints and fix resizability as needed.

I'm a bit curious what this will do on Windows, too. :)

Note You need to log in before you can comment on or make changes to this bug.