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

update #1

Open
wants to merge 4,105 commits into
base: master
Choose a base branch
from
Open

update #1

wants to merge 4,105 commits into from

Conversation

ZwoVierZwo
Copy link

No description provided.

zaikos and others added 30 commits January 14, 2015 13:28
…o get the proper title tag from a feed item.
Updated FeedItem_RSS::get_title()
… a default namespace so let's add a separate ugly hack for rdf:RDF feeds, thanks for that xml dipshits
Previously the floating title would not appear until the real title
scrolled entirely out of view, which was visually distracting.  Now it
appears at the point when its bottom lines up with the real title’s
bottom.  This isn’t perfect yet due to small differences between the
two titles’ layouts, but it’s much closer.

We now need to use visibility: visible|hidden instead of display:
block|none to show and hide the floating title, in order to be able to
retrieve its height while it’s hidden.  It’s important to use that
height because occasionally line wrapping differences make the
floating title shorter than the real title, and we don’t want the real
title to poke out from underneath.

Also fix scrollToRowId to offset by the 4 pixel top margin (like
cdmScrollToArticleId already does), so that it goes back to a point
when the floating title was hidden; and set the right margin on the
floating title so that it’s the same width as the real title and
doesn’t overlap the scroll bar.

Signed-off-by: Anders Kaseorg <[email protected]>
Make the floating title transition smoother
Conflicts:
	locale/de_DE/LC_MESSAGES/messages.po
Andrew Dolgov and others added 26 commits July 13, 2015 19:24
Various updates and some typo fixing
Updated Italian translation
Updated messages.mo for Italian translation
tribut pushed a commit to tribut/Tiny-Tiny-RSS that referenced this pull request Nov 25, 2015
andre-hub pushed a commit to andre-hub/Tiny-Tiny-RSS that referenced this pull request Mar 6, 2021
andre-hub pushed a commit to andre-hub/Tiny-Tiny-RSS that referenced this pull request Jul 8, 2022
andre-hub pushed a commit to andre-hub/Tiny-Tiny-RSS that referenced this pull request Jul 17, 2022
If I enabled some user plugins via Preferences -> Plugins and then
disabled all of them, an error occurred:

Jul 16 22:34:20 php[100]: PHP Fatal error:  Uncaught TypeError: array_filter(): Argument torne#1 ($array) must be of type array, null given in /usr/share/webapps/tt-rss/classes/pref/prefs.php:1027
Jul 16 22:34:20 php[100]: Stack trace:
Jul 16 22:34:20 php[100]: #0 /usr/share/webapps/tt-rss/classes/pref/prefs.php(1027): array_filter()
Jul 16 22:34:20 php[100]: torne#1 /usr/share/webapps/tt-rss/backend.php(136): Pref_Prefs->setplugins()
Jul 16 22:34:20 php[100]: torne#2 {main}
Jul 16 22:34:20 php[100]:   thrown in /usr/share/webapps/tt-rss/classes/pref/prefs.php on line 1027

Apparently the issue was elevated from a warning to an error in PHP 8.0
[1].

[1] https://php.watch/versions/8.0/internal-function-exceptions
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.