Thank you, I really appreciate it!
Thatâs because by default, iPads request the desktop site but iPhones donât. Users can always request desktop/mobile site, so if this is the only thing different between the two, you might not want to trust that.
You could add a date picker if you wanted to add a mobile-friendly / more precise input for the snapshot or add a checkbox to show the snapshot on tapping so it wonât appear unless the user wants it toâŠ
I just tried out 3.6.0 and projections finally appeared on my iPad.
However, the screenshot shows a problem with the level progressions graph. When I tap in the grey area above level 17, the input for the median line covers the entire thing, so I canât look at the details for that.
Would there be some way you could have an option to just show as text that the median is 10d 18h (or whatever) and remove the line from the graph⊠or some other option that would make it possible to see the permanently invisible information?
@saraqael Iâve made a pull request on Github with some styling changes/fixes, mostly for dark mode (although also a couple of other small things such as alignments and the header menu). Iâve tested it and I donât think Iâve broken anything!
No worries if you donât want to merge it for any reason though!
Wow, this really looks great!
Thank you so much for taking the time to do this, I really appreciate it. How couldnât I merge this? I really like the rounded edges (I thought of doing this way back but because I overdid it it didnât look that good, but this has a completely different appearance). Great job!
The only small thing I noticed was this:
In dark mode the bars look greyed out. But in my opinion it looks better than when they are not, so Iâm just going to leave it like this
And I think I will also apply that rounded look to the settings boxes to make them fit the style
Ah yes, good point, I didnât think to round the settings boxes!
I also noticed the lack of colour in the chart, but wasnât really sure how to fix it. The problem with leaving it greyed out is that it might not be obvious that itâs a stacked chart if you arenât familiar with it. So Iâd probably recommend either fixing the colours or adding a title to that chart to specify what it is!
If I get some time in the next few days I might make another pull request to make the website look a bit better on large monitors
Youâre right even though it doesnât look that bad it can be confusing!
You really donât have to. But if you insist who am I to intervene? Thanks!
What is this piece of code supposed to do? Because without it the problem seems fixed. It changes colors of exactly these âgâ element to grey (#d6d6d6).
.dark-mode g > g > g > rect:only-child {
fill: #d6d6d6;
}
.dark-mode g > g > g > rect:nth-child(1):nth-last-child(2) {
fill: #d6d6d6;
}
.dark-mode g > g > g > rect:nth-child(2):nth-last-child(1) {
fill: #d6d6d6;
}
Which chart should this format? Because itâs not my codeâŠ
That CSS is to make the edge grid lines on the charts white instead of gray or black, so that they stand out more (equivalent to the thicker black lines in light mode).
Not sure why itâs affecting anything else though, Iâll have a look now!
EDIT: Worked out how I messed it up, working out a fix now
Fixed, Iâll submit a pull request in a minute!
I found a way around this to additionally check for iPads and it seems to work for my model.
Then I think I know what the mistake was. The dates that your userscript passed on were in a different weird format that depends on the device and the string wasnât recognized as a date again when I wanted to translate it back. This was also the problem on one of my devices. I changed one thing so that your userscript passes the dates directly which presumably fixed the issue.
I did it like this (the checkbox should only appear on mobile).
Added a hide median line feature.
Edit: Oops, just realized currently itâs completely useless because the hiding is only visual:
I implemented this. The order is also saved.
And a bonus feature: total averages. So the averages over the whole existence time span of the user:
@saraqael Just in case you donât get email notifications, I submitted a PR #4 to your repo.
Nice:) Although I did encounter a bug at once: I canât bring the ććçèȘ up past the âWall of Xâ section.
As for the one day snapshot what I had in mind was more something like this:
where you can see at a glance whether a high or a low accruacy score was a really good/bad performance or if it was just due to a low number of items.
I also just find this table hard to read
I can see that 100% accuracy on radicals was just from answering 1 review correctly, while 100% on kanji was from 16 reviews and 98.8% was from 79 vocab reviews, so it does make it easier than having to scroll between different graphs to make the comparison but itâs also just densly packed with all this other data with similar looking names Not that more info is a bad thing but donât we allready get all this presented in a clearer manner in the info table?
Also: I donât get what the average column is showing (especially for âReviews Radicalâ). Average over how many days?
Might a reading coverage chart like the one on WKStats fit in somewhere (like the right hand column)?
Thatâs because on desktop there are different columns (left and right) for certain items on desktop which shouldnât mingle with each other. But on mobile because there isnât as much space horizontally it is just put below it. But due to the html structure Iâve used for the columns it would be difficult and weird to have this only be switchable on mobile. (If you are on desktop you also have this behavior if your screen is less than 1800px wide; if you zoom out with Control + â-â you will also see the right side bar.)
That is already on my to-do list, Iâm on it!
This is the average over all the days you were active, so the âtotal averageâ basically. So â5.3â is the average amount of radical reviews you would do on any given day where you were doing wanikani.
I thought so too, but this is what I interpreted that overview to be. Maybe Iâll change it up a bit to be more usable.
I saw that just now and for me it seems to have created the projections span again at the level chart:
Can I delete that element or should I just hide it? Because for me it seemed like deleting it didnât cause any problemsâŠ
I thought about something similar (obviously I donât want to just copy it from WK stats). But because the amount of charts has gotten quite big, I might split the site into sub pages. And one of these sub pages could be something like Reading Coverage where you can look at charts of how much percent of certain kanji collections one can already read (also N5-N1 but additionally others like maybe genki or even famous Japanese books) just to get an idea how far one has gotten with their Japanese journey and to motivate them. Because it is sometimes surprising how fast one can actually learn kanji on WK.
Sounds good! Should I hold off on making stuff look better on wide screens for now then if youâre going to split the site into sub pages? (in case you change the number of columns or something) Or arenât you going to split it quite yet?
If Iâm done implementing the current list of requests/problems Iâm going to start with the sub pages. But that could take a little while so you definitely can work on the design (thank you again for that)! Either way I wonât change the style of the website too much so I could then just apply the new style to the sub pages also.
@saraqael how would you feel about moving the menu bar with the links up to the top in line with the log-out button and version number something like this? (havenât done any styling yet, wanted to check what you thought first)
I feel like theyâre not nearly as important as the internal links to the different parts of the page and so they just take up unnecessary space and attention having a whole large nav bar to themselves!
I thought maybe the menu bar could be the sub page navigation? But I can worry about that later. The links on there were actually just placeholders in the beginning but then I just kept them. Although I think itâs time to completely remove the old website link because it is not really that important. But yeah, I think this looks better!
Would changing the settings boxes to appear on hover like this be fine with you? I think it would help declutter the page a bit!
Youâre really working hard on this! The page already looks miles better than before. So yeah, I really like that look.