Conversation
|
The others should be marked as TODO. The CLI tool can be used to automate that. |
|
Thanks 👍🏻 @marienfressinaud A lot of it seems to be quite obsolete. Furthermore, there are other, more important packages we use and that are not in the list. See https://github.com/FreshRSS/FreshRSS/blob/edge/README.md#included-libraries Maybe we should remove this string and move the remaining relevant things to the readme. |
marienfressinaud
left a comment
There was a problem hiding this comment.
This makes totally sense 👍
@marienfressinaud I do not think this is relevant anymore, is it?
@marienfressinaud I am not sure how much of that is left. Would you know?
This font is still used, but we have a couple more used by our different themes. |
I've always read it as that the general concept of |
|
Thanks for the feedback. I've pushed some changes to move the credits to the README and delete them from the web UI. |
MINZ (archived, read-only) is not a dependency of FreshRSS, which would be very concerning. Instead, FreshRSS copied MINZ and has evolved it since then under the same license. Ref: FreshRSS#7425
The look and colors were also very inspired by Bootstrap 2. You can compare the buttons at https://getbootstrap.com/2.3.2/base-css.html#buttons for example. I wrote the CSS myself though, so I'm not sure it's pertinent to keep the reference to Bootstrap (especially as it changed a lot since then).
I don't think the icons have changed much, so I'd say they all still come from GNOME (for instance, the "read view" icon)
I found Lato, Open Sans and Spectral under https://github.com/FreshRSS/FreshRSS/tree/edge/p/themes/fonts @glyn I noticed that you re-introduced the credit to Minz in the README, you can remove it :) |
|
A few more changes applied |
|
With @Alkarex's changes, I believe this PR is ready to merge. Please advise. /cc @marienfressinaud, @Frenzie |
|
Let's go for that for now, and do more changes in another PR if needed. |
|
@glyn Please add a line for you in https://github.com/FreshRSS/FreshRSS/blob/edge/CREDITS.md |
Done! Thanks @Alkarex. |
MINZ (archived, read-only) is not a dependency of FreshRSS, which would be very concerning.
Instead, FreshRSS copied MINZ and has evolved it since then under the same license.
Ref: #7425
Move credits
This PR moves the credits from the web UI to the README.