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

Minimap occupies full viewport height even if it's shorter than the viewport height #337

Closed
mnquintana opened this issue Jun 5, 2015 · 2 comments
Labels

Comments

@mnquintana
Copy link

(This is a feature suggestion, not a bug)

screenshot 2015-06-04 22 45 46

It would be awesome if there were an option (or a default) for a "floating mode" where the minimap "floats" above the editor instead of blocking text at a certain width (not sure how coherent this is – I should probably get some sleep 😆)

@abe33 abe33 added the feature label Jun 5, 2015
@abe33
Copy link
Contributor

abe33 commented Jun 5, 2015

Hi @mnquintana, Yes it was requested several time now, so I guess it's probably time that I integrate it. I once proposed a CSS fix for that in that #322 that looked like this:

atom-text-editor::shadow atom-text-editor-minimap {
  position: absolute;
  right: 0;
}

@abe33
Copy link
Contributor

abe33 commented Jun 5, 2015

Ok, I didn't fully understood your proposal at first. I think pure floating (as a floating div do in a regular html page) will be impossible to achieve, because it would require to have different wrapping length for each line based on the height of the minimap. But an absolute positioning should achieve pretty much the same effect. Let me know how you feel about that.

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