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

Sync to latest Sidebar (2016-04-16) #138

Merged
merged 12 commits into from
Apr 25, 2016
Merged

Sync to latest Sidebar (2016-04-16) #138

merged 12 commits into from
Apr 25, 2016

Conversation

flatcap
Copy link
Contributor

@flatcap flatcap commented Apr 16, 2016

Changes in Sidebar since NeoMutt 20160404.
They include

  • Several important bug fixes
  • A couple of slight changes to behaviour
  • Some updates to the docs

flatcap and others added 12 commits April 16, 2016 14:46
Sidebar calculated the message counts slightly differently from Mutt.
This occasionally lead to the counts changing unexpectedly.
Respect the TITLE_FMT when on the compose screen.
Sidebar didn't redraw properly when in the pager, or help.
If a user deletes the current mailbox with "unmailboxes", but then adds
it back with "mailboxes", then we will highlight it correctly.
Certain folder settings, combined with sidebar_folder_indent, caused an overflow
when trying to compare paths.

    set folder="~/mail/folder"
    mailboxes "~/mail/folder/"

Note the trailing / on the mailbox name.

Signed-off-by: Richard Russon <rich@flatcap.org>
Originally '$sidebar_divider_char' could be an arbitrary string.

When I reworked Sidebar I limited it to 1 character (because I couldn't
see a good case for anything else).

It turns out that ' | ' isn't a bad choice for the divider.

fixes: #15
@flatcap flatcap changed the title Issue/138 Sync to latest Sidebar (20160416) Apr 16, 2016
@flatcap flatcap changed the title Sync to latest Sidebar (20160416) Sync to latest Sidebar (2016-04-16) Apr 16, 2016
@karelzak karelzak merged commit 65f5c52 into karelzak:master Apr 25, 2016
@flatcap flatcap deleted the issue/138 branch July 24, 2016 16:27
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.

3 participants