Skip to content

Conversation

@loufranco
Copy link
Contributor

I based this visualization on the retention heat map, but simplified it so I could use it on my more generic 2-label data.

I don't know what you use to minimize, but I could add a commit with the correct minimizations if you tell me how to do it.

@loufranco
Copy link
Contributor Author

@leqilong What is the best way to move this PR forward? I can add the min.js if you tell me how -- I could make any other changes you want.

I'm using this now in a mode report and it would be nice if I could just include it (which requires it to be hosted by mode)

@leqilong
Copy link
Contributor

leqilong commented Nov 1, 2019

Hi @loufranco, sorry for missing your PR. We use uglifyjs to minimize the alamode.js file. You can use this command to do so:
uglifyjs --compress --mangle -- alamode.js

As for the alamode.css file, we use csso. And the command is:
csso alamode.css --output alamode.min.css

@leqilong leqilong requested review from leqilong and removed request for leqilong November 1, 2019 19:27
@loufranco
Copy link
Contributor Author

Ok @leqilong, I minified. Let me know if there's anything else I need to do.

@leqilong leqilong merged commit ba1da84 into mode:master Nov 1, 2019
@leqilong
Copy link
Contributor

leqilong commented Nov 1, 2019

@loufranco Thanks. I've deployed your changes. You should be able to start using it now.

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