Skip to content
This repository has been archived by the owner on Sep 30, 2021. It is now read-only.

"No newline at end of file" could be replaced with an icon like on the website #31

Open
wasdennnoch opened this issue Feb 27, 2017 · 12 comments

Comments

@wasdennnoch
Copy link

wasdennnoch commented Feb 27, 2017

No description provided.

@k0shk0sh
Copy link
Owner

could you let me where is this line is located? I'm abit lost with this one.

@wasdennnoch
Copy link
Author

How it's on desktop (I have a dark theme but the red little icon is the same): http://puu.sh/umbVW/04f9d3d98a.png

vs how it's on mobile: http://puu.sh/umbXS/d68e700448.png

@Maxr1998
Copy link

Any updates? I also think this would look a lot cleaner.

@k0shk0sh
Copy link
Owner

@Maxr1998 pass me the same exact icon in svg format and you can get it by next release :D

Sent from my Htc m8 using FastHub

@Maxr1998
Copy link

Maxr1998 commented May 29, 2017

There you are. I just extracted the icon svg from the GitHub html and converted it to Android, thus I'm not yet sure whether you're allowed to use it..

EDIT: Actually, you are: https://octicons.github.com/icon/no-newline/

<vector xmlns:android="http://schemas.android.com/apk/res/android"
    android:width="16dp"
    android:height="16dp"
    android:autoMirrored="true"
    android:viewportHeight="16.0"
    android:viewportWidth="16.0">
    <path
        android:fillColor="#ff0000"
        android:pathData="M16,5v3c0,0.55 -0.45,1 -1,1h-3v2L9,8l3,-3v2h2V5h2zM8,8c0,2.2 -1.8,4 -4,4s-4,-1.8 -4,-4 1.8,-4 4,-4 4,1.8 4,4zM1.5,9.66L5.66,5.5C5.18,5.19 4.61,5 4,5 2.34,5 1,6.34 1,8c0,0.61 0.19,1.17 0.5,1.66zM7,8c0,-0.61 -0.19,-1.17 -0.5,-1.66L2.34,10.5c0.48,0.31 1.05,0.5 1.66,0.5 1.66,0 3,-1.34 3,-3z" />
</vector>

@k0shk0sh
Copy link
Owner

@Maxr1998 this probably will cause crashes on Android 21 due to the negatives in the svg :D . I have copied some from GitHub but then the penalty was crashes lol. I'll have to test this icon on Android 21 to 26 😕

Sent from my Htc m8 using FastHub

@Maxr1998
Copy link

Well, at least the converted icon doesn't use the SDK23+ fillType attribute - let's see if all api levels support it then ;)

@k0shk0sh
Copy link
Owner

Sent from my Htc m8 using FastHub

@Maxr1998
Copy link

Awesome work, thanks!

@k0shk0sh
Copy link
Owner

@Maxr1998 thank you for the icon :)

@yakov116
Copy link
Collaborator

yakov116 commented Jul 28, 2017

@k0shk0sh the icon is not aligned correctly when it's on the old code side.

Link to commit

sriharshaarangi/BatteryChargeLimit@d287e21

Sent from my Google Nexus 9 using FastHub

@yakov116 yakov116 reopened this Jul 28, 2017
yakov116 pushed a commit that referenced this issue Aug 8, 2017
@yakov116 yakov116 removed this from the 1.4.0 milestone Sep 13, 2017
@cozyplanes
Copy link
Contributor

Done?

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

No branches or pull requests

6 participants