A bit more info in the notification emails

zeeguu / api

API for tracking a learner's progress when reading materials in a foreign language and recommending further personalized exercises and readings.

https://zeeguu.org

MIT License

8 stars 24 forks source link

A bit more info in the notification emails #214

Closed mircealungu closed 2 months ago

mircealungu commented 2 months ago

Email notifications with included article titles are cool!

One thing that we see above is that the same topic appears in two articles. I guess it would be nice if we showed the respective: sources, difficulty ratings, and lengths? It would better distinguish the two, and show the learner that they have options!

mircealungu commented 2 months ago

Possible formattings:

Similar to what you used but with bold title to increase legibility

kina = = =

Slut med at gå på pension som 50-årig: Kina hæver pensionsalderen (2min, B1) on dr.dk

Kina hæver pensionsalderen til 58 år for kvinder og 63 år for mænd (4min, A2) on politiken.dk

Title more readable without the underline

... but the repetitive read link is annoying

kina = = =

Slut med at gå på pension som 50-årig: Kina hæver pensionsalderen (2min, B1) read on dr.dk

Kina hæver pensionsalderen til 58 år for kvinder og 63 år for mænd (4min, A2) read on politiken.dk

tfnribeiro commented 2 months ago

I actually think I don't mind the Read repetition, I think it also makes you look at the source link so you know where you will go to.

But with the title I was actually thinking that it might be nice to just use an h3 or similar, e.g.

Kina

Slut med at gå på pension som 50-årig: Kina hæver pensionsalderen (2min, B1) read on dr.dk

Kina hæver pensionsalderen til 58 år for kvinder og 63 år for mænd (4min, A2) read on politiken.dk

mircealungu commented 2 months ago

I like the H3 idea for the subject!

mircealungu commented 2 months ago

and with H3, metadata on the second row, and bold title?

Kina

Slut med at gå på pension som 50-årig: Kina hæver pensionsalderen 2min, B1, read on dr.dk

Kina hæver pensionsalderen til 58 år for kvinder og 63 år for mænd 4min, A2, read on politiken.dk

tfnribeiro commented 2 months ago

That looks good - I do it!

Do you want the line separating the H3 from the articles or is it fine to just use paragraphs as spacing?

mircealungu commented 2 months ago

Less is more? :)

mircealungu commented 2 months ago

Note, the abusing of    to indent the metadata :)

tfnribeiro commented 2 months ago

Something like this?

The HTML doesn't read the paragraphs as it does in python, but the articles should be the same: