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

Broken styling for git-view if placed on the left of the status bar #65

Closed
UrsaDK opened this issue Mar 15, 2017 · 10 comments
Closed

Broken styling for git-view if placed on the left of the status bar #65

UrsaDK opened this issue Mar 15, 2017 · 10 comments
Labels

Comments

@UrsaDK
Copy link

UrsaDK commented Mar 15, 2017

I'm using move-status-items plugin and noticed that dragging .git-view on to the left of the status bar (inside .status-bar-left) breaks a lot of the styling.

Would be nice if .git-view could be placed on either side of the status bar...

@fv0 fv0 added the Bug label Apr 3, 2017
@fv0
Copy link
Owner

fv0 commented Apr 8, 2017

@UmkaDK Could you provide a screenshot?

@UrsaDK
Copy link
Author

UrsaDK commented Apr 8, 2017

Sure thing, will do! But I'm away at the moment, so it might take me a few hours... Would tomorrow be ok?

@UrsaDK
Copy link
Author

UrsaDK commented Apr 8, 2017

Got back a bit earlier then expected, so ... There are three screen shots attached to this post:

  1. a view of the broken styling when .git-view is on the left of the statusbar
  2. the same as 1 but with Dev Tools open
  3. the same as one but with my fixes applied

Broken Styling:

broken-styling

broken-with-devtools

Locally, I managed to fix the issue by adding this to my styles.less:

status-bar {
    // TEMP: A temporary fix for native-ui theme
    @import "ui-variables";
    @import "status-bar";
    .status-bar-left {
        margin-left: 0;

        .git-view {
            padding: 0 @status-bar-spacing;
            padding-left: 0;
            margin: 0 3px;
            margin-right: 0;

            .inline-block {
                color: @text-color-selected;
                padding: 0 @status-bar-spacing;
                padding-right: 0;
                margin-right: 0;
            }
        }
        > .inline-block {
            margin: 0 0 0 3px;
            padding: 0 @status-bar-spacing;

            &:hover {
                background-color: darken(@tab-background-color, 5%);
                border-radius: @border-radius--base;
                text-decoration: none;
                color: white;
                cursor: pointer;

                a,
                span {
                    color: white;
                }
            }
        }
    }
}

which, when applied, looks like this:
my-fix

@fv0
Copy link
Owner

fv0 commented Jun 3, 2017

@UmkaDK How’s your issue going with latest update?

@UrsaDK
Copy link
Author

UrsaDK commented Jun 5, 2017

Just downloaded and installed version 0.21.0, disabled all custom styles and ... the spacing appears to be a lot better, but as you can see, branch name is still showing up as black on black. :(

screen shot 2017-06-05 at 13 53 12

@fv0
Copy link
Owner

fv0 commented Jun 6, 2017

@UmkaDK That’s weird. Maybe you can answer these questions for me.

  1. Have you got the latest version of Atom installed?
  2. Does the text color of the branch change when you’re in a repo that has modifications?
  3. What does the gray symbol stand for? Maybe you can inspect this and show me the HTML code structure for this part.

@UrsaDK
Copy link
Author

UrsaDK commented Jun 6, 2017

  1. Atom version 1.17.2 (as of this moment, it's the latest available stable release)
  2. In a repo that has modifications the colour of the branch name doesn't change (it remains dark on dark), the colour of the branch status text does change (to orange-ish).
  3. The grey symbol indicates that I was editing a file which is ignored by git. :)

status-bar-inspect

Sorry for a bit of a terse reply. A little bit swamped with work at the moment. If there is anything else I can do, please let me know. It might take me a day or two to reply but I'm happy to help.

@UrsaDK
Copy link
Author

UrsaDK commented May 17, 2018

Just thought I would update this for the latest Atom / Native-UI:

  • Atom: 1.27.0
  • Native-UI: 0.26.1

As mentioned in the original message (at the top), dragging github status display to the left of the status bar (inside .status-bar-left) breaks some of the styling, ie:

broken

Adding the following CSS to your .atom/styles.less resolves the issue:

.theme-native-ui {
  .status-bar-left {
    .github-StatusBarTileController {
      background-color: inherit;
      margin-right: 11px;
      &:hover {
        background-color: #a6a6a6 !important;
      }
    }
  }
}

and updates github status display to look like:

fixed

@fv0 fv0 closed this as completed in 4df62b3 May 20, 2018
@fv0
Copy link
Owner

fv0 commented May 20, 2018

@UmkaDK Finally fixed this. Can you confirm with version 0.26.2 of Native UI that this solves your problem? It didn’t make sense to scope the styling to status-bar-right, so I removed this and adapted the code to handle the new Fetch link in the Git status element.

@UrsaDK
Copy link
Author

UrsaDK commented May 21, 2018

@fv0, almost there!! :)

The colours are now spot on, the text is readable, everything works good. The only problem is the padding: there is too much of it on the left and not enough on the right, see attached screen shot:

v0 26 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants