-
Notifications
You must be signed in to change notification settings - Fork 7
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
time instead of authors? #326
Comments
@mircealungu, I like the idea! Also, I have observed that (so far) it only happens in the reader from the app, the reader in the extension can extract the authors. I will check more articles to confirm if though. If so, maybe comparing parsing solutions in both cases could help fix it 🤔 |
That's a good point Iga, we parse the documents slightly differently from the Extension VS the Browser, so maybe this would be a problem that would fix itself once we unify the two readers. |
I think it's very likely. But I also like the solution of showing the source logo and a date (before we figure out why the author is not extracted correctly). If unifying two readers fixes the parsing issue, I would still leave the source logo present. |
we are not extracting the authors correctly from some articles. happens at least in Politiken (e..g https://www.zeeguu.org/read/article?id=2455030) and DR (e.g. https://www.zeeguu.org/read/article?id=2445554).
as a result, the reader can be confusing at times:
Also, we're not showing the article date in the reader.
I propose the following solution:
what do you think @igawaclawska and @tfnribeiro ?
The text was updated successfully, but these errors were encountered: