Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make #2452 feature with adjust column width to text more customizable #2586

Merged
merged 4 commits into from
Dec 22, 2024

Conversation

akruphi
Copy link
Contributor

@akruphi akruphi commented Dec 19, 2024

Yet more after #2452

image
image

@akruphi akruphi changed the title Adjust column width to "folder" text only if in mode it width = 0 Make #2452 feature with adjust column width to text more customizable Dec 20, 2024
@elfmz
Copy link
Owner

elfmz commented Dec 22, 2024

А можно подробнее что это исправляет?
Ну и до кучи - лучше уж чекбокс, "Always adjust 'Size" column width" с пояснением в хелпе

@akruphi
Copy link
Contributor Author

akruphi commented Dec 22, 2024

А можно подробнее что это исправляет? Ну и до кучи - лучше уж чекбокс, "Always adjust 'Size" column width"

В #2452 была внесена фича, что колонка Размер всегда (даже если в её размер задан не ноль, а фиксированное число) подстраивается под ширину текста у каталогов - вот этим фрагментом

else {
if (ColumnType == SIZE_COLUMN || ColumnType == PHYSICAL_COLUMN) {
if (ColumnWidths[I] < ColumnTypeWidth[SIZE_COLUMN])
ColumnWidths[I] = ColumnTypeWidth[SIZE_COLUMN];
}

Это вызывает совершенно неожиданное расширение этой колонки, например
image
при этом исторически в настройках размера стоит 6,
image
которое в таком случае напрочь игнорировалось.

Я сделал эту фичу настраиваемой - по умолчанию приоритет ширины из настройки столбцов с обрезкой этих текстов (как классически все и привыкли и как сделано в far3), но внутри Ctrl+Alt+D вместе с выбором текста каталогов/симлинков в колонке Размер можно переключится на подстройку ширины столбца по тексту, как закодено в #2452.

с пояснением в хелпе

По всему этому окну Ctrl+Alt+D написать отдельный помощи как-нибудь руки дойдут. Но @anta999 в тг-чате вроде согласился, что возможно расширит там варианты надписей более чем текущие варианты
image
- поэтому подождём пока они устаканятся прежде чем писать помощь.

@elfmz elfmz merged commit 4a251fb into elfmz:master Dec 22, 2024
9 checks passed
@akruphi akruphi deleted the size_column branch December 22, 2024 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants