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

adjust color scale to use it fully #35

Closed
wants to merge 1 commit into from

Conversation

dancek
Copy link

@dancek dancek commented Feb 19, 2015

I implemented automatic color scale adjustment so that the color scale is fully used every time. This is the default behavior in my version, but you can disable it using the -G or --static-colors parameters.

Examples

k

screen shot 2015-02-20 at 01 45 26

k -G

screen shot 2015-02-20 at 01 45 54

@dancek
Copy link
Author

dancek commented Feb 19, 2015

I think my version is much better for file sizes, as different directories tend to have different file sizes so you rarely get good use of the color scale.

For dates, it's not perfect. Dates don't seem to conform to a nice pattern; often you have most files from about the same time, and maybe a couple of newer ones, so often you only get about two colors.

@supercrabtree
Copy link
Owner

Coooool!! This is a way better idea

@zirrostig
Copy link
Contributor

I definitely like the file sizes change on this. But I'm not sure about the dates. Could you possible make it so I could use just one or the other maybe something along the lines of

-G      --static-colors=WHICH            do not adjust color scale automatically
                                  WHICH=files,dates,both

obviously format that to be clearer. zparseopts should be pretty staightforward to do that with.

Otherwise looks great.

@dancek
Copy link
Author

dancek commented Apr 29, 2015

In these past two months I've forgotten all about the code. I'm also way too busy with our newborn twins. Hopefully someone else can pick this up.

@zirrostig
Copy link
Contributor

zirrostig commented Apr 29, 2015 via email

@jirutka
Copy link

jirutka commented May 17, 2015

+1

@dancek
Copy link
Author

dancek commented Dec 21, 2023

Closing as obsolete.

@dancek dancek closed this Dec 21, 2023
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.

4 participants