Skip to content
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

feedly theme not shown in TT-RSS 18.12 preferences #72

Closed
lvii opened this issue Feb 14, 2019 · 7 comments
Closed

feedly theme not shown in TT-RSS 18.12 preferences #72

lvii opened this issue Feb 14, 2019 · 7 comments

Comments

@lvii
Copy link

lvii commented Feb 14, 2019

Hi

I have running tt-rss 18.12 in docker with last version linuxserver/tt-rss :

# docker inspect -f '{{ index .Config.Labels "build_version" }}' tt-rss
Linuxserver.io version:- 18.12-ls2 Build-date:- 2019-02-12T14:36:56+00:00

I have copied css files to tt-rss/themes.local/ dir following README.md :

$ ls -lhF /config/www/tt-rss/themes*
/config/www/tt-rss/themes:
total 12K
-rw-rw-r-- 1 abc abc  328 Nov 30 22:02 compact.css
-rw-rw-r-- 1 abc abc  192 Nov 30 22:02 default.php
-rw-rw-r-- 1 abc abc 4.0K Nov 30 22:02 night.css

/config/www/tt-rss/themes.local:
total 64K
drwxr-xr-x 3 abc abc 4.0K Feb 14 13:31 feedly/
-rw-r--r-- 1 abc abc  60K Feb 14 13:31 feedly.css

$ md5sum /config/www/tt-rss/themes.local/*.css
471016e09edef84ee934f7d179689237  /config/www/tt-rss/themes.local/feedly.css

$ find /config/www/tt-rss/themes.local
/config/www/tt-rss/themes.local
/config/www/tt-rss/themes.local/feedly.css
/config/www/tt-rss/themes.local/feedly
/config/www/tt-rss/themes.local/feedly/fonts
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-bold-webfont.woff
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-regular-webfont.ttf
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-regular-webfont.woff
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-bolditalic-webfont.woff
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-italic-webfont.woff
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-italic-webfont.ttf
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-bolditalic-webfont.ttf
/config/www/tt-rss/themes.local/feedly/fonts/texgyreheroscn-bold-webfont.ttf

There was no feedly theme in TT-RSS preferences :

img

@ltguillaume
Copy link

ltguillaume commented Feb 18, 2019

Same here... Tried the release and f9555c4

@levito
Copy link
Owner

levito commented Feb 18, 2019

Strange... I have no explanation. Have you tried putting the files into themes instead of themes.local? This would be the old way.

Also, can you ensure that the dockerized ttrss version is 18.12 and not older?

@ltguillaume
Copy link

Already tried. Putting the files into themes/ doesn't work either.
I'm not using the Docker version, but my test installation is quickly set up via Softaculous, footer states Tiny Tiny RSS v18.12

@oblogic7
Copy link

I just ran into the same issue. Have confirmed that it is not a file permission issue.

image

@ltguillaume
Copy link

ltguillaume commented Feb 19, 2019

Adding the following line to all css files got the themes to show in the Preferences:
/* supports-version:18.12 */

However, this was the result:
image

Simply put, your theme does not support v18.12. It only supports the current master branch. As soon as I updated from the Git repo, the theme started working.

@lvii
Copy link
Author

lvii commented Feb 19, 2019

Hi @levito

TT-RSS v18.12 was no lib/flat-ttrss/ dir :

https://git.tt-rss.org/fox/tt-rss/src/4db888c7c281d13d9e2d0a5b9f0fb3aa063e8842/lib

lib/flat-ttrss/ dir was only in current master branch :

https://git.tt-rss.org/fox/tt-rss/src/master/lib/flat-ttrss

feedly.css first line @import url(../lib/flat-ttrss/flat_combined.css); imported a not existing file.

@levito
Copy link
Owner

levito commented Feb 19, 2019

Sorry for the inconvenience, ttrss versioning is kind of broken. There was a short period of time when everything was changing rapidly and the version was already at 18.12. If you are not seeing this theme in the preferences, you are probably using this state of ttrss code. Please update from ttrss master then. Since ttrss uses rolling release, that's the official way to go anyway.

@levito levito closed this as completed Feb 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants