When running the dialog with a theme that has no window decoration it is not possible to close it because the "upper right x" is hidden. Every dialog needs means such as confirmation buttons Ok/Cancel to close. As an easy solution the toolbar of this dialog could have another button with the action TBI_CLOSE. The ui file is located at svx/uiconfig/ui/imapdialog.ui.
Incorrect. From the General Keyboard Shortcut help: Esc:: Terminates the action or dialog. If in LibreOfficeDev Help: goes up one level. So we already provide the <Esc> shortcut, does that not function? IMO this UI behavior is similar to what we've now done for the <Alt> key to toggle mnemonic accelerators visible. Why in general clutter each dialog with additional buttons--just to accommodate those choosing "a theme that has no window decoration"?
The issue was reported by a user and she had no clue how to close the dialog (not sure if escape works on her system). Additionally, we aim for consistency and familiar layout is a goal, and accessibility - escape is typically connected to the cancel button, setting a modal result without an UI control might be hackish.
I initially had no idea how to close the dialog either (in the absence of upper right corner [x]) Most of our dialogs, even modeless ones, usually have some Cancel/Close/Dismiss button, so this should be made consistent. That said, I'd not do that through toolbars, but add some regular button to the bottom of the dialog ... but different people can have different opinions on this. Thus, confirmed. Inherited from OOo since it's always been like that
Well it raises an interesting question--is the project responsible for providing GUI elements that an OS and DE removes? Guess we do it for OS X so why not each Desktop flavor for Linux? Otherwise adding the _additional_ button widget for close/cancel to each dialog is intrusive, since the frame decoration already has the button and when ommitted by the DE the <Esc> short-cut performs the function anyhow. Not a fan.
Abhilash Singh committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bc8eb82877f544d8cf80e6f3ea02a9993e058491 tdf#103574 Imagemap dialog lacks on confirmation button It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Assume this is resolved
(In reply to jan iversen from comment #6) > Assume this is resolved Yes, it is. Still one the ugliest dialog, and the close button somewhere in the toolbar is also not the best UX. But the functionality is there now. Thanks Abhilash!