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

Code minimap #40

Open
cdelahousse opened this issue Feb 5, 2013 · 4 comments
Open

Code minimap #40

cdelahousse opened this issue Feb 5, 2013 · 4 comments

Comments

@cdelahousse
Copy link
Member

Imported issue by spratt from 2012-06-26T14:42:18Z
Originally opened as SteamedPears/CodeReviewClientMaster#7


Sublime text has a "code minimap" at the left side of the screen which shows the code at a very high level. This is kind of gimicky, but if we highlight the lines with comments then it can help finding comments in large blocks of code.

@ghost ghost assigned bheesham Feb 5, 2013
@cdelahousse
Copy link
Member Author

Imported comment by Gankro from 2012-07-15T05:20:30Z *


The more I think about this, the less excited I am about it. However if we do this I demand a "comment/diff" heat map indicating activity.

@cdelahousse
Copy link
Member Author

Imported comment by spratt from 2012-07-15T05:33:49Z *


I really like the idea of being able to see at a glance and move directly to wherever there are comments in a big file. A comment/diff heatmap is a must.

@spratt
Copy link
Member

spratt commented Feb 5, 2013

Peter had the idea of vertical lines alongside the code indicating comments on the sections of the code next to the lines.

Code_Review1
Code_Review2

@bheesham
Copy link
Member

bheesham commented Feb 6, 2013

Yeah, the minimap doesn't seem like such a good idea with the new proposal for the layout.
I do like the idea of the colour-coded lines.

But as someone brought up in the email, what about colour blind people? Unless we consider them an edge case we don't have to worry too much about.

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

No branches or pull requests

3 participants