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.

Sign up for GitHub

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

Jump to bottom

Cannot set mini-format for "brief" listing type #3588

Closed
mc-butler opened this issue Jan 24, 2016 · 6 comments
Closed

Cannot set mini-format for "brief" listing type #3588

mc-butler opened this issue Jan 24, 2016 · 6 comments
Assignees
@aborodin
Labels
area: core Issues not related to a specific subsystem prio: low Minor problem or easily worked around
Milestone
4.8.16

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3588
Reporter mooffie (@mooffie)

Steps to reproduce:

  1. Go to the "Listing mode" dialog.
  2. Select "Brief file list".
  3. Tick the "User mini status" checkbox.
  4. Modify the mini status format field (e.g., append " inode").
  5. Hit ENTER. Bug: the format you modified doesn't take effect.

The patch fixes this.

(Note: if you close the dialog by hitting ENTER when the focus is at the radios widget, the modified mini-format won't take effect even with this patch. This is because of a non-related bug fixed in #3587 (search there for "it overwrites the mini status format the user has typed").)

Note

Original attachments:
@mc-butler
Copy link
Author

Changed by mooffie (@mooffie) on Jan 24, 2016 at 14:57 UTC

@mc-butler
Copy link
Author

Changed by zaytsev-work (@zyv) on Jan 25, 2016 at 8:29 UTC (comment 1)

  • Milestone changed from Future Releases to 4.8.16

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 28, 2016 at 13:29 UTC (comment 2)

  • Branch state changed from no branch to on review
  • Votes set to andrew_b
  • Status changed from new to accepted
  • Owner set to andrew_b

Branch: 3588_mini_format
[2347ac3]

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 28, 2016 at 13:30 UTC (comment 3)

  • Branch state changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 28, 2016 at 13:31 UTC (comment 4)

  • Votes changed from andrew_b to committed-master
  • Status changed from accepted to testing
  • Resolution set to fixed
  • Branch state changed from approved to merged

Merged to master: [2347ac3].

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 28, 2016 at 13:32 UTC (comment 5)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aborodin aborodin

Labels
area: core Issues not related to a specific subsystem prio: low Minor problem or easily worked around
Milestone
4.8.16
Development

No branches or pull requests
2 participants
@aborodin @mc-butler