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

css scale #4

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

whytheplatypus
Copy link

Glad this extension exists!

here's a version that uses css transforms... not sure if it's better or worse... but it was a fun little project :)

@drewhamlett
Copy link
Owner

Hey man this is great. Thanks. I'm working on getting this in.

@drewhamlett
Copy link
Owner

Hi, if it's not to much trouble, can you look into the new version using CodeMirror.runMode and implement with that? Thanks!

@whytheplatypus
Copy link
Author

oh good idea, i hadn't seen that. Yeah I'll check it out tomorrow and see what i can get running

@whytheplatypus
Copy link
Author

There we go, I think that should work. It even made some of the scaling issues a little better.

@drewhamlett
Copy link
Owner

Hey man I like this concept but I think we need a fixed font size for the mini-map. If the length of the document changes(is really short) the font size is really big. Is it doing this for you?

@whytheplatypus
Copy link
Author

Yeah it is. The options I can think of are: get rid of the scale and go back fixed font sizes, or have some desired width for the map and use a scaling ratio that shrinks it to that width if the document is short.

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.

2 participants