[PATCH] Mouse click on column heading sorts panel on that column

[bnh]

Attached patch that adds the following 2 enhancements:
1) Using the mouse to click on a panel column heading re-sorts

the panel on that column. If already sorted on that column,
the sort order is reversed.

2) added a button "." next to "v" that toggles the display

of hidden files.

[bnh]

Patch to screen.c to allow sorting on panel columns via mouse click

[andrew_b]

Replying to bnh:

2) added a button "." next to "v" that toggles the display

of hidden files.

It's a semi-solution. In addition to mouse, the shortcut should be provided.
The length of panel title (current directory) should be corrected also.

[slavazanko]

reated branch 397_mouse_click_column_heading

initial changeset:dac1c1c21fcff330582c4d66cd13dcbd2fab45ac

review, please.

[slavazanko]

It's a semi-solution. In addition to mouse, the shortcut should be provided.

In this case this branch wil continue to develop after megre #212 into master.

[slavazanko]

See changeset:45c0931f997eeddb327a7defe9a8ffa48ae2c59e

Added function and key binding for toggle sort types.

For review, you may change mc.keymap:

[panel]
...
-PanelToggleSortOrder =
+PanelToggleSortOrder = <some-hotkey>

Review, please.

[slavazanko]

Fix toogle sort after select sort type from dialog.
changeset:1c65f3bb72c77779cfa50630c7e707b43aa2be61

[slavazanko]

Work complete.

• 3ca63869bbca682555596e3c42fe3e697996d5a6 : Added support of mouse events
• 4fb134450cf2e85c53c2fe71f904e8044b188f19 : Some fixies from Andrew Borodin
• b0e01b7458bb2d58c0994a491fbe4da3e34a5c26 : reorganization of panel fields
• ca5555fc70aefae84c675df9eff95b93e896f1b0 : Remove hardcoded array of field names from listmode.c
• b41243a8be4a6b41db36c9da90ba36d167e59af7 : Added event handlers for select or toggle of sort types. Key bindings not provide (user may bind hotkeys to events as well).

For example, you may change mc.keymap like:

[panel]
PanelSelectSortOrder= alt-w
PanelToggleSortOrderPrev=alt-e
PanelToggleSortOrderNext=alt-d

Feel free to review and vote.

After vote branch will have 'on hold' status until release out 4.7.0-pre3.

[iNode]

PanelSelectSortOrder does not change sort order and sometimes show incorrect sort order.

PanelToggleSortOrderPrev and PanelToggleSortOrderNext works good (but order of changes looks strange).

[slavazanko]

Now all fixed. See:

• 627496125636981e223613d822ecfd1e8b374564: Remove unneded code
• 1e51fff77ceb6464ec9b2de206b89505a822f4ae: Fixed behavior of CK_PanelSelectSortOrder event
• b2c601d2dceb590600ba73a172df2307057a0195: Remove code for reverse search direction when toggle search types.
• 10f3a940b5517d3df3df8bfab31457b17adf2759: Added keymaps for some sort types. Also added added 'PanelReverseSort?' keymap.

Review, please.

[slavazanko]

new commits into branch:

• a26db6491c14bd95a15ef0732789bc86c8d77fcf: Added reversing sort order ability if one keybind raised multiple times.
• 0d185fa2b8d3df30663ab6240d8c79eede92f383: Fixed segfault in 'Brief file list' listing mode
• 640e67fc2cfbe11e58bd3ca11a06a489bf66cc61: Added indicator for current sort type and sort direction
• 1420e7d4bdc631711a9269bb884cce1cdf511ba5: Cleanup of code. Remove unused panel_field_t::use_in_gui variable

Review.

[slavazanko]

merge changeset:77896d5de17c79baaf929497be42dbf379903520

[dmartina]

I'm not sure if using the &Hotkey for the indicator in the top left corner is the best choice. Hotkey is choosen for it's uniqueness in dialog and sometimes it is not quite representative.
For example, I tkink 'A' is the proper indicator for ATime, but I can't make it hotkey as "tiempo de &Acceso" because I'm already using it in "&Aceptar" for "OK". I suppose there may be similar troubles in other languages. I would let translators decide, an issue some TRANSLATORS messages in .po files in order to warn them about this use and not using more than a single character.

Something else: in screen.c line 1329, panel_paint_sort_info()

hotkey[1] = '\0';

doesn't seem to me very UTF-8 safe.

[slavazanko]

I'm not sure if using the &Hotkey for the indicator in the top left corner is the best choice.
Hm... Is you have alternative proposal? I think about this and I found variants:

• show sort indicator as is in English (for example, show first char from field name)
• show sort indicator as one translated char.

In second variant need to inform all translators about one symbol in *.po files (what this mean).

How better? Is someone have third variant?

Something else: in screen.c line 1329, panel_paint_sort_info()
hotkey[1] = '\0';
doesn't seem to me very UTF-8 safe.

All ok. in previous line:

1328         hotkey = g_strdup(panel->current_sort_field->id);

'id' will not transform into UTF-8 and never translated. This is id of field. For example, see line 436, 448 etc. In these lines 'id' - it's a first string constant.

[slavazanko]

Reopen because need to fix indicator of sort types.

[slavazanko]

Created branch 397_i18n_sort_indicator

Initial changeset:7088ee899ae8c1a85ae74b49ee7c3689c3aa7c95

Review, please.

[iNode]

It's ugly solution that violate DRY principle, and require
from translators read sources, but what's done is done.

[slavazanko]

Changeset for rebase:6a3b517a955d07c44377b3626c9e362bd4fdc97d

dmartina, if this not hard for you: review too, please. Is branch contain good way?

iNode: is you have solution?

[dmartina]

Replying to slavazanko:

I'm not sure if using the &Hotkey for the indicator in the top left corner is the best choice.
Hm... Is you have alternative proposal? I think about this and I found variants:

• show sort indicator as is in English (for example, show first char from field name)
• show sort indicator as one translated char.

I would vote for the second one.

In second variant need to inform all translators about one symbol in *.po files (what this mean).

See this in args.c. The comment gets into po files so that translators don't have to read source.
#. TRANSLATORS: don't translate keywords and names of colors
#: src/args.c:309
The comment has to be in the previous line, before the string marked for translation.

How better? Is someone have third variant?

Something else: in screen.c line 1329, panel_paint_sort_info()
hotkey[1] = '\0';
doesn't seem to me very UTF-8 safe.

All ok. in previous line:

1328         hotkey = g_strdup(panel->current_sort_field->id);

'id' will not transform into UTF-8 and never translated. This is id of field. For example, see line 436, 448 etc. In these lines 'id' - it's a first string constant.

I trust you!

[dmartina]

Replying to slavazanko:

Changeset for rebase:6a3b517a955d07c44377b3626c9e362bd4fdc97d

dmartina, if this not hard for you: review too, please. Is branch contain good way?

iNode: is you have solution?

I have no time to compile and full check it now... I took a look at the patch and it seems OK

TRANSLATORS: this is indicator for 'Change time' sort mode

I would remark the one character length. Something as :
TRANSLATORS: one single character to represent 'Change time' sort mode

[slavazanko]

TRANSLATORS: one single character to represent 'Change time' sort mode

changeset:d88182b5d99e1b2ab7cfec93591f288885527ba9

All ok. in previous line:

I trust you!

This code now removed as deprecated. :)

See this in args.c. The comment gets into po files so that translators don't have to read source.
#. TRANSLATORS: don't translate keywords and names of colors

Yepp, this works as well in my patches too.

After merge branch into 'master', I'll update all *.po files directly into master (to avoid lot of possible conflicts in merge).

[slavazanko]

merge changeset:11ec99633325b84c9601159101ba4c7072ae6f0d

Also, updated translations: de72a6f98f2b9c60d55c16e638b73bbd3b5328b7

[slyfox]

There is missing line in one of structs:

gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../../mc/src -I..  -DDATADIR=\""/home/slyfox/dev/git/mc-build/_mc-bin/share/mc/"\" -DLOCALEDIR=\""/home/slyfox/dev/git/mc-build/_mc-bin/share/locale"\" -DSAVERDIR=\""/home/slyfox/dev/git/mc-build/_mc-bin/libexec/mc"\" -DSYSCONFDIR=\""/home/slyfox/dev/git/mc-build/_mc-bin/etc/mc/"\"  -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include   -I../../mc  -g -O2 -Wall -MT screen.o -MD -MP -MF .deps/screen.Tpo -c -o screen.o ../../mc/src/screen.c
../../mc/src/screen.c:470: warning: initialization makes integer from pointer without a cast
../../mc/src/screen.c:470: error: initializer element is not computable at load time
../../mc/src/screen.c:470: error: (near initialization for 'panel_fields[4].use_in_user_format')
../../mc/src/screen.c:472: warning: initialization from incompatible pointer type

[slyfox]

Replying to slavazanko:

merge changeset:11ec99633325b84c9601159101ba4c7072ae6f0d

Also, updated translations: de72a6f98f2b9c60d55c16e638b73bbd3b5328b7

msgstr "Без сортировки"" 

Too many trailing ", it won't build:

cd ../../mc/po && rm -f ru.gmo && /usr/bin/gmsgfmt -c --statistics -o ru.gmo ru.po
ru.po:2684: символ конца строки встречен внутри строки
/usr/bin/gmsgfmt: найдена 1 критическая ошибка

[andrew_b]

Replying to slyfox:
< {{{

ru.po:2684: символ конца строки встречен внутри строки
/usr/bin/gmsgfmt: найдена 1 критическая ошибка
}}}

Fixed directly in master. changeset:e2e549d4bc529e174548784d09e9af2e5623497d

[slavazanko]

created branch 370_fix_init_struct

initial changeset:55fe1bb86e68d447818844025372888467619211

Review, please.

Sorry for inconsistence :(

[slavazanko]

merge changeset:07b9ae3a26fe3bd323458aad09d893e482c4da85

[dmartina]

If asking for one character translations may seem a bit weird, a context may be given. I'm attaching the patch. Anyway, I don't dislike the actual solution. Advice for translators should stay in any case.

[dmartina]

Use context for indicator translations.

[andrew_b]

In this ticket, I dislike the double i18n stuff. For example:

N_("Unsorted"), N_("&Unsorted"),
N_("Name"), N_("&Name"),
N_("Extension"), N_("&Extension"),

I wonder, is there any way to avoid that?

[dmartina]

Replying to andrew_b:

In this ticket, I dislike the double i18n stuff. For example:

N_("Unsorted"), N_("&Unsorted"),
N_("Name"), N_("&Name"),
N_("Extension"), N_("&Extension"),

I wonder, is there any way to avoid that?

Trim any '&' after runtime translation. But keep in mind that these single word translations may cause trouble when used in different contexts (ie.: different shortcuts or even available length in a menu or in a dialog), so more use of the Q_ macro would be quite convenient.

[dmartina]

I have tried to change the sort indicators to have up/down arrows instead of tildes and it looks really nice. ¿What would be the best choice? ¿System or user skins? I think it might at least be included as part of the double-lines skin.

[slavazanko]

I think it might at least be included as part of the double-lines skin.

Hm... Totally I agree with this. 'Standart' skin must be untouched (or must contain very simple skin for show on many systems as this possible). In opposite, 'double-lines' skin will contain enhanced skin as example of usage.

I think, need to rename 'double-lines' skin into 'featured'... or need to create new repo-inside file... what better? :)

[slavazanko]

I created tickets:

• #1714 - Sort types: use context for indicator translations.
• #1715 - Sort types: duplicate of i18n strings
• #1716 - Adding of fully featured skin

Therefore this ticket is closed.
Don't reopen, please :) Create new tickets instread.