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

Problem with tooltip regarding last update time #31

Closed
sleek22 opened this issue Sep 9, 2013 · 5 comments
Closed

Problem with tooltip regarding last update time #31

sleek22 opened this issue Sep 9, 2013 · 5 comments
Assignees
Labels

Comments

@sleek22
Copy link

sleek22 commented Sep 9, 2013

There was a bug in Brief 1.7a1 that was carried into Digest that was not in Brief 1.6.2 and it has to do with the count since last update time (the code for the tooltip time count was changed between these two versions). For example, if feeds are set to update each hour, the count will increment but as soon as the system clock crosses the hour mark, the tooltip will show the feeds updated one hour ago and stay that way. The timing is still accurate but does not show on the tooltip.

To see what I mean, hit the update button at ten minutes before the top of the hour. It will count up to ten minutes, immediately jump to one hour, and stay that way for the next fifty minutes.

The link below shows my reporting the bug and the author of Digest fixing it:

tanriol/digest#39

Since the author of Digest now recommends that we move back to Brief,and the bug is still there in Brief 1.7 which I just tested, could you please look into this problem as the fix in Digest worked perfectly.

@ghost ghost assigned tanriol Sep 9, 2013
@tanriol
Copy link
Member

tanriol commented Sep 9, 2013

Thanks for reminder, I forgot to port this fix as it needs some refactoring. Will do this soon.

@ancestorak
Copy link
Collaborator

@tanriol, what's the status?

@tanriol
Copy link
Member

tanriol commented Oct 4, 2013

@ancestorak, oh, I forgot about this one. In a few days.

tanriol added a commit to tanriol/brief that referenced this issue Oct 8, 2013
This fixes brief-rss#31. For the sake of uniformity all consumers are migrated to
the new functions.
@tanriol tanriol mentioned this issue Feb 23, 2014
@ancestorak
Copy link
Collaborator

@tanriol, this was fixed by #44, correct?

@tanriol tanriol closed this as completed Oct 10, 2014
@tanriol
Copy link
Member

tanriol commented Oct 10, 2014

Yes, should be fixed. Closing.
Apologies for a trash closing message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants