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

[patch] color tag viewnormal not accepted #3404

Closed
mc-butler opened this issue Feb 16, 2015 · 5 comments
Closed

[patch] color tag viewnormal not accepted #3404

mc-butler opened this issue Feb 16, 2015 · 5 comments
Assignees
@aborodin
Labels
area: skin Theming support and skin files prio: medium Has the potential to affect progress
Milestone
4.8.14

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3404
Reporter and

color tag viewnormal is not respected, because
underlying bsearch() expected a sorted array to work correctly

attached patch will solve this bug.

Note

Original attachments:
@mc-butler
Copy link
Author

Changed by and on Feb 16, 2015 at 15:27 UTC

old_color table sort fix

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 18, 2015 at 6:41 UTC (comment 1)

  • Owner set to andrew_b
  • Milestone changed from Future Releases to 4.8.14
  • Version changed from 4.8.13 to master
  • Branch state changed from no branch to on review
  • Component changed from mc-core to mc-skin
  • Status changed from new to accepted
  • Votes set to andrew_b

Thanks! Applied.

Branch: 3404_old_colors_sort
[29d4afc]

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 18, 2015 at 6:43 UTC (comment 2)

  • Branch state changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 18, 2015 at 6:43 UTC (comment 3)

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

Merged to master: [29d4afc].

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 18, 2015 at 6:46 UTC (comment 4)

  • 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: skin Theming support and skin files prio: medium Has the potential to affect progress
Milestone
4.8.14
Development

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