Bug 64357 - UI: A whole new tab was created in a dialog to host a single check box (follow-up to bug 42195)
Summary: UI: A whole new tab was created in a dialog to host a single check box (follo...
Status: RESOLVED INVALID
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
4.1.0.0.alpha0+ Master
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: needsDevEval
Depends on:
Blocks:
 
Reported: 2013-05-08 12:34 UTC by Adolfo Jayme Barrientos
Modified: 2015-12-18 10:27 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:

Attachments
Okular with font property (72.27 KB, image/png)
2013-08-01 01:56 UTC, Manik Chand 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adolfo Jayme Barrientos 2013-05-08 12:34:43 UTC 
I’ve just saw this in the 4.1 release notes wiki page: https://wiki.documentfoundation.org/images/9/92/Embed_font.png

This is a huge waste of space and unnecessary UI and code complexity added to the “Properties” dialog. There’s absolutely no need to add a new .ui file to create a new tab page that will just have one tiny check box in it, that’s very silly.

That check box, “Embed fonts in the document”, could have been placed in the “General” tab instead.
Comment 1 Olivier R. 2013-05-08 12:55:54 UTC 
I agree.
Comment 2 João Paulo 2013-05-26 02:27:35 UTC 
Also agree.
Comment 3 Milan Bouchet-Valat 2013-05-31 12:34:19 UTC 
The checkbox could probably go to the General tab.
Comment 4 Adolfo Jayme Barrientos 2013-05-31 18:36:10 UTC 
@Milan: Yeah, that’s what I said :-)
Comment 5 João Paulo 2013-07-31 05:26:50 UTC 
I think the checkbox could have a child checkbox for subsetting the font. If this clutters the General tab, maybe the Security and Fonts tabs could be merged.
Comment 6 Manik Chand 2013-08-01 01:56:05 UTC 
Created attachment 83410 [details]
Okular with font property

Okular has property dialog with font tab. They have used it to show embedded fonts.
Comment 7 Joel Madero 2014-02-27 22:55:27 UTC 
In order to limit the confusion between ProposedEasyHack and EasyHack and to make queries much easier we are changing ProposedEasyHack to NeedsDevEval.

Thank you and apologies for the noise
Comment 8 Adolfo Jayme Barrientos 2015-01-05 16:20:16 UTC 
In the meantime, I have added a heading label to this checkbox, so that the tabpage doesn’t look confusingly empty anymore. Also, this (and other dialogs) are being improved in many ways, so this bug report has become obsolete. Resolving INVALID as we don’t have the OBSOLETE status. ;-)

I think adding the list of fonts Manik suggested would be nice, but that requires another bug report.
Comment 9 Robinson Tryon (qubit) 2015-12-18 10:27:18 UTC 
Migrating Whiteboard tags to Keywords: (needsDevEval)
[NinjaEdit]