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

tt-rss 17.1 #36

Merged
merged 3 commits into from
Feb 15, 2017
Merged

tt-rss 17.1 #36

merged 3 commits into from
Feb 15, 2017

Conversation

talklittle
Copy link
Contributor

Some of the CSS selectors have changed. And the Special feeds now have their own container, so I updated the CSS for that.

I couldn't figure out how to change the large 30px margin on the left side of the feeds tree, where the collapse button used to be. Sidebar collapse functionality in tt-rss has been moved to a new plugin (toggle_sidebar), whose button appears in the main feed area, not in the feed tree area.

@TheFiZi
Copy link

TheFiZi commented Jan 23, 2017

Thank you. Works like a charm.

@disconn3ct
Copy link

Working here also, thanks!!

@x86dev
Copy link

x86dev commented Jan 23, 2017

Thanks for the patch!

@thob
Copy link

thob commented Jan 24, 2017

Working here too. Except I got an issue with long feed titles in the leftmost column. If too long I need to scroll horizontally to see the new items counter.

@Takika
Copy link

Takika commented Jan 24, 2017

@thob: I have the same problem :(

@talklittle
Copy link
Contributor Author

Anyone want to take a stab at fixing the problem reported by @thob ? It's probably related to the 30px padding problem I mentioned in the PR description. Seems like Dojo/Dijit are doing some layout in the JS, I can't figure out how to properly override it in CSS.

@Takika Takika mentioned this pull request Jan 27, 2017
@usr42
Copy link

usr42 commented Jan 30, 2017

This branch is also working for me. Thanks!

@Takika
Copy link

Takika commented Jan 30, 2017

I found this commit in main TT-RSS git, but I don't know how to implement this in feedly:
https://tt-rss.org/gitlab/fox/tt-rss/commit/24a82359b2a190e276f95d5cd53ed78940912d4e

I think enough to add this to the end of feedly.css:

body#ttrssMain #feedTree.dijitTree .dijitTreeContainer {
	max-width : 100%;
}

body#ttrssMain #feedTree.dijitTree .dijitTreeRow {
	overflow: hidden;
	text-overflow: ellipsis;
}

@levito
Copy link
Owner

levito commented Jan 30, 2017

Hi guys, I just moved and am about to start a new job. Right now, I don't have a development machine. I'll look into this in a couple of days, stay tuned! Sorry for letting you wait so long.

@talklittle
Copy link
Contributor Author

@Takika Great find! I'll update the PR. Thanks.

copied from tt-rss upstream commit

24a82359b2a190e276f95d5cd53ed78940912d4e
@usr42
Copy link

usr42 commented Jan 30, 2017

@talklittle The fix for the long titles is working for me. The counter is always visible.
Thanks!

@levito levito merged commit 77335f0 into levito:master Feb 15, 2017
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

Successfully merging this pull request may close these issues.

None yet

8 participants