"Chmod" dialog: Use screenspace better (with patch)

[mc-butler]

Important

This issue was migrated from Trac:

Origin	https://midnight-commander.org/ticket/2557
Reporter	bugreporter

MC's "chmod" dialog has its size hardcoded in the source -- both the overall size, and the size of the dialog's components.

The hardcoded numbers work well with the English language (see the attached screenshot "english_before.png").

They do NOT work well with many other languages (see the attached screenshot "international_before.png"; it uses German as an example). Perfectly translated strings overflow their allocated width. At the same time, the dialog leaves much blank space that stays unused.

The attached patch "mc-4.7.5.2-chmod_use_whitespace-1.patch" re-structures the "chmod" dialog's internal dimensions such that less of the dialog's area is wasted for whitespace. The dialog's outer dimensions stay untouched.

This improves the situation for non-English languages considerably (see the attached screenshot "international_after.png").

The dialog in English language still looks good (see the attached screenshot "english_after.png").

Note

Original attachments:

• english_before.png (raw) by bugreporter on May 16, 2011 at 17:39 UTC
• english_after.png (raw) by bugreporter on May 16, 2011 at 17:39 UTC
• international_before.png (raw) by bugreporter on May 16, 2011 at 17:40 UTC
• international_after.png (raw) by bugreporter on May 16, 2011 at 17:40 UTC
• mc-4.7.5.2-chmod_use_whitespace-1.patch (raw) by bugreporter on May 16, 2011 at 17:41 UTC
• mc-4.7.5.2-chmod_use_whitespace-2.patch (raw) by bugreporter on May 16, 2011 at 22:23 UTC
• international_before_and_after.png (raw) by bugreporter on May 17, 2011 at 17:28 UTC

[mc-butler]

Changed by bugreporter on May 16, 2011 at 17:39 UTC

• Attachment english_before.png (raw) added

[mc-butler]

Changed by bugreporter on May 16, 2011 at 17:39 UTC

• Attachment english_after.png (raw) added

[mc-butler]

Changed by bugreporter on May 16, 2011 at 17:40 UTC

• Attachment international_before.png (raw) added

[mc-butler]

Changed by bugreporter on May 16, 2011 at 17:40 UTC

• Attachment international_after.png (raw) added

[mc-butler]

Changed by bugreporter on May 16, 2011 at 17:41 UTC

• Attachment mc-4.7.5.2-chmod_use_whitespace-1.patch (raw) added

[mc-butler]

Changed by bugreporter on May 16, 2011 at 22:23 UTC

• Attachment mc-4.7.5.2-chmod_use_whitespace-2.patch (raw) added

[mc-butler]

Changed by bugreporter on May 16, 2011 at 22:23 UTC (comment 1)

The blank before the "[ ]" boxes is actually used for something, so it must stay. An updated patch is attached: "mc-4.7.5.2-chmod_use_whitespace-2.patch".

[mc-butler]

Changed by andrew_b (@aborodin) on May 17, 2011 at 8:35 UTC (comment 2)

• Resolution set to wontfix
• Status changed from new to closed

chmod dialog is not only one where the i18n is implemented not quite correctly. Unfortunately, your patch also contains hardcoded widget positions. Try select several files in panel and then open chmod (or chown) dialog. You see that position of buttons are not well. I think, buttons should be located in two lines for this case.

In general, i18n of chmod and chown and many other dialogs should be reimplemented totally.

As a result, I'd close this ticket as wontfix because of #64.

[mc-butler]

Changed by andrew_b (@aborodin) on May 17, 2011 at 8:36 UTC (comment 3)

• Keywords chmod dialog size, i18n, usability, professional-level software quality, attention to detail deleted

[mc-butler]

Changed by bugreporter on May 17, 2011 at 17:28 UTC

• Attachment international_before_and_after.png (raw) added

[mc-butler]

Changed by bugreporter on May 17, 2011 at 17:29 UTC (comment 4)

• Keywords set to chmod dialog size, i18n, usability, professional-level software quality, attention to detail
• Status changed from closed to reopened
• Resolution wontfix deleted

Reopening.

These buttons overlap with and without the patch. The patch does not introduce that overlap. See the attached screenshot "international_before_and_after.png".

The patch does not solve all of the chmod dialog's problems. But it improves the current situation -- without introducing additional code, and without undesired side-effects. Please provide a better reason why you don't want it.

[mc-butler]

Changed by andrew_b (@aborodin) on May 18, 2011 at 10:35 UTC (comment 4.5)

Replying to bugreporter:

The patch does not solve all of the chmod dialog's problems.

And why not fix all problems of one dialog at once?

Please provide a better reason why you don't want it.

Because this is a semi-solution of chmod i18n problems. Again, why not fix a chmod dialog completely at once?

[mc-butler]

Changed by bugreporter on May 18, 2011 at 18:18 UTC (comment 5.6)

Replying to andrew_b:

And why not fix all problems of one dialog at once?

The changes that are needed to fix all problems, and properly (from the ground up), are not trivial. They go beyond my current knowledge of MC, glib, and even C. What I can do is to find and fix things like missed opportunities (that's what the patch does), off-by-one errors, and so on. The boring groundwork.

And that is important, too. By replacing the existing set of hard-coded values with a better set, my trivial patch removes a lot of blatant ugliness in the non-English interface (see the screenshots). We must still wait for someone to do the big rewrite, but at least we have a more usable international interface while we wait.

The same can be done for the buttons in the chmod dialog (to spread them out horizontally a bit better), and perhaps also for other dialogs. But before doing more work it is interesting to see whether such semi-solutions have a chance of getting accepted upstream at all.

[mc-butler]

Changed by gms (gms_mouse@….ru) on May 20, 2011 at 12:19 UTC (comment 7)

Hi.
I've seen a fork of MC with solved i18n dialog buttons problem. The buttons were placed in 2 rows in chmod and chown dialogs. But I've forgotten what a fork it have been :(

[mc-butler]

Changed by andrew_b (@aborodin) on May 20, 2011 at 12:25 UTC (comment 7.8)

Replying to gms:

I've seen a fork of MC with solved i18n dialog buttons problem. The buttons were placed in 2 rows in chmod and chown dialogs. But I've forgotten what a fork it have been :(

I have my own patches for that.