-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
move vue and vue-calendar-heatmap to npm/webpack #10188
Conversation
There are more Vue warnings visible when webpack builds in development mode. I want to eventually enable us to use that mode, but right now I can not get bindata-less builds to work at all for some reason which is a requirement for that. |
e207386
to
9b3a311
Compare
Codecov Report
@@ Coverage Diff @@
## master #10188 +/- ##
==========================================
- Coverage 43.69% 43.68% -0.02%
==========================================
Files 586 586
Lines 81354 81386 +32
==========================================
+ Hits 35547 35552 +5
- Misses 41410 41431 +21
- Partials 4397 4403 +6
Continue to review full report at Codecov.
|
The heatmap should probably be moved to a |
@lafriks any feedback? If you like, I can try porting heatmap to a |
Looks good, it can be converted to Vue in other pr |
c085ef9
to
dd8016b
Compare
Rebased. This is good to go from my perspective. |
Okay, I can do that part. Added to todo. |
@gary-kim I'd prefer if we create a single vue rendering function to be used by all the vue components. I think the current code initializes vue in 2 or 3 different places, which seems like useless code duplication to me. Maybe create |
I like that idea. Make a single rendering function that imports all the Vue components and calls |
Rebased |
Any more reviews here? |
- unvendor vue and vue-calendar-heatmap - remove unused moment.js leftover from previous heatmap version - ensure webpack loads the full version of vue - fix vue devmode warning related to 'searchLimit' type I wanted to name the chunk heatmap.js but adblockers don't like that filename [1]. [1] https://github.com/easylist/easylist/blob/3899d5dff33216c0bc64f09ff15d376f346d3e33/easyprivacy/easyprivacy_general.txt#L2095
Rebased once more, good to go. |
I wanted to name the chunk heatmap.js but adblockers don't like that filename [1].
[1] https://github.com/easylist/easylist/blob/3899d5dff33216c0bc64f09ff15d376f346d3e33/easyprivacy/easyprivacy_general.txt#L2095