Skip to content

Tabs do not pick dark theme on Mojave #775

Closed

Description

@pointlessone

Tabs do not fully pick up dark theme in Mojave. Only text changes color. This results in a really low-contrast look. It wouldn't be much issue if it didn't change text color but currently it make it hard to use MacVim tabs with dark theme.

This is most probably related to #544.

screen shot 2018-11-10 at 5 06 41 pm

Activity

  1. ychin commented on Nov 15, 2018

    @ychin
    Member

    Are you building MacVim locally or using HomeBrew? Right now Mojave doesn't work very well due to #751 and you should see a lot of flickering when you use MacVim built for Mojave. Currently focus is on fixing that first, but I will look into the tab issues.

  2. pointlessone commented on Nov 15, 2018

    @pointlessone
    Author

    I use Homebrew binary.

    I know about #751. I have Core Text Renderer disabled. It works (doesn't flicker) apart from minor space width issues on otherwise blank lines.

  3. bhargavshivkumar commented on Nov 16, 2018

    @bhargavshivkumar

    Is there a work around for this until #751 is resolved?

  4. ychin commented on Nov 17, 2018

    @ychin
    Member

    @bhargavshivkumar You could do one of the following:

    1. Use the snapshot version in https://github.com/macvim-dev/macvim/releases (they are all built against High Sierra instead of Mojave for now).
    2. Build MacVim using Xcode 9 yourself. I don't know if Homebrew provides a method to do that.
    3. Don't use Dark Mode (which I understand is not a real solution)

    Edit: Also, this issue is unrelated to #751 so even if that issue is resolved this issue would still persist.

  5. bhargavshivkumar commented on Nov 20, 2018

    @bhargavshivkumar

    @ychin When you say use the snapshot version i'm assuming you are saying i should install using the dmg file instead of using homebrew binaries? (sorry relatively new to the Mac way of things)
    Also, is there a specific snapshot I should be downloading or is 151 also built against High Sierra?

  6. ychin commented on Nov 20, 2018

    @ychin
    Member

    @bhargavshivkumar Yes I meant installing the prebuilt dmg file from Github releases (we just call them snapshots). The latest one (151) should be fine and is built against High Sierra.

    I plan to merge in all Mojave support soon so Homebrew builds won't stay broken. There should be a corresponding snapshot-152 release as well and it would be built against Mojave.

  7. added 4 commits that reference this issue on Nov 24, 2018
    713b64f
    3d91f12
    7982284
    6e1489e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions