You can subscribe to this list here.
| 2003 |
Jan
(4) |
Feb
(22) |
Mar
(18) |
Apr
(9) |
May
(20) |
Jun
(2) |
Jul
(14) |
Aug
|
Sep
|
Oct
|
Nov
(3) |
Dec
(112) |
|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 2004 |
Jan
(94) |
Feb
(18) |
Mar
(49) |
Apr
(22) |
May
(18) |
Jun
(4) |
Jul
(9) |
Aug
(3) |
Sep
(4) |
Oct
|
Nov
|
Dec
(2) |
| 2005 |
Jan
(2) |
Feb
|
Mar
(15) |
Apr
(1) |
May
(2) |
Jun
|
Jul
|
Aug
|
Sep
|
Oct
(1) |
Nov
|
Dec
(1) |
| 2006 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
|
Jul
(2) |
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
| 2007 |
Jan
|
Feb
(1) |
Mar
|
Apr
|
May
|
Jun
|
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
| 2008 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
|
Jul
|
Aug
|
Sep
|
Oct
(1) |
Nov
|
Dec
|
| 2009 |
Jan
|
Feb
(2) |
Mar
|
Apr
(2) |
May
|
Jun
|
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
|
From: Nicole Simon <nisi@gn...> - 2003-05-03 23:08:47
|
"Kristjan Jansen" <kristjan.jansen@...> wrote: >I'm glad someone steps up to improve the usability of the program, finally :) *grin* i think we will like each other ;o) As a keyboard-shortcut-junkie I have several things to moan about being non standard ms ;o)) So, let's start with the most annoying parts: Number one on my hitlist: If I add a new feed via F2, I always have it's address in the clipboard and paste it. Pressing Enter let's FR check if it is ok and if it is - I have to use the mouse to finish it - just because someone does not follow the basic principle of windows software: most possibly non destructiv possibility is choosable by enter. Just put a focus at "Finish" and every one will be happy. (remember, i'm new and do new feeds all the time) Number two: I don't like IE. And I don't want IE. And I "can't use" it because I have made it's security as high as possible. You probably have done it becuase only IE can display in FR but please give me the choice to say "no, I don't want it, open new window in favorite browser" which should only be a dde-command as far as I remember. Number three: It would be nice if F5 Update would mean update this feed (as in a Webpage where you also update this page and not every page your browser has opended) and not everything. Everything should be sth special and get Ctrl-F5. Number four I do already have a lot of feeds and like them sorted in folders. Because of all this jumping up and down of the folders when i move an entry I would like to have a Ctrl-F2 "Make new in this folder" so I can choose folder first and get it set to the right place Number five comes with #1: If you paste an address to new feed and it is by accident (which happens more than sometimes here) " url" or "url " it get's an error. Space is not an allowed char for url - how about a stripspaces(newfeed)? Number six: I would expect the 'jump to next unread' would means 'next unread in view'. Does this jump to "take the next feed which was added after this and jump to next unread there'? If I do scroll in the preview pane, ins is not working (of course not). btw ins is not really a good choice for 'go to next unread' - and i would like to have an alt shortcut ... :o) And I have to admit - what's with the favorites in FR? Oh, did I mention that I do like it besides this and some other facts? ;o) Nicole -- Nicole Simon · nisi@... · http://home.pages.de/~nisi/ Hardware, n.: The parts of a computer system that can be kicked. |
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-05-03 22:31:17
|
> (btw - i only saw a webbased mailarchive, is there a=20 > downloadble one or > someone who could provide me a zipped .mbox archive? so I=20 > know what was > already discussed? thx) I guess you have to stick with the webarchive: http://sourceforge.net/mailarchive/forum.php?forum_id=3D20340 Check out my postings "Review of FR user interface 1-4". I'm glad someone steps up to improve the usability of the program, = finally :) Kristjan |
|
From: Nicole Simon <nisi@gn...> - 2003-05-03 22:23:30
|
Hi :o) I'm new to this list and like to introduce myself. Nicole Simon, 32 years, from Marzipan-City, Northern Germany. Addicted to Feeds for two days and not so addicted on Feedreader cause there are many things which drive me nuts (mainly things with baaaad usabilty in this little piece). ;o) (btw - i only saw a webbased mailarchive, is there a downloadble one or someone who could provide me a zipped .mbox archive? so I know what was already discussed? thx) >What community thinks. Is there a need for menu in Feedreader? There are Yes. Every windows program should have one even if it just has some menuentrys in it. >already a lot of possible buttons, that do not need to be in toolbar: You have to have them in toolbar now cause you don't have a proper menu where you could choose from. So if you just have occasionally the feeling of using ... Nicole -- Nicole Simon · nisi@... · http://home.pages.de/~nisi/ Hardware, n.: The parts of a computer system that can be kicked. |
|
From: Toomas Toots <tom@i-...> - 2003-05-03 15:20:11
|
What community thinks. Is there a need for menu in Feedreader? There are already a lot of possible buttons, that do not need to be in toolbar: About Properties News Overview (this is Marcus'es secret weapon ;)), press F7 in last release. More to come... If all these are on toolbar, then toolbar looks quite unclean. Greetings Toomas |
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-04-28 14:49:36
|
> 1. If feed has no dc:date field, then datetime is generated by=20 > feedreader and it will be current datetime. Problem is, that=20 > if there is=20 > 10 new headlines for example, then they all get the same=20 > datetime value=20 > and if they are displayed in listview, then sort order is=20 > quite weird.=20 They display on a reverse order, you mean? Or totally mixed up? Kristjan |
|
From: Toomas Toots <tom@i-...> - 2003-04-28 11:00:43
|
There are two bugs in Feedreader, that need to be addressed. 1. If feed has no dc:date field, then datetime is generated by feedreader and it will be current datetime. Problem is, that if there is 10 new headlines for example, then they all get the same datetime value and if they are displayed in listview, then sort order is quite weird. It would be nice to have the last headline on the top and so on... I currently do no have any ideas, how to fix this. Maybe we should add milliseconds to datetime field internally by feedreader? After all this date field is interpreted as string in feedreader. 2. There is a problem with ExpandEscapes function what is used by html2normal function, for example. If we give html to html2normal : ---------------Example from http://www.scripting.com/rss.xml--------------- <a href="http://rss.com.com/2100-1025-998465.html?type=pt&part=rss&tag=feed&subj=news">News.Com</a>;: "The Webby Awards, the Internet community's biggest annual event, has become a non-event this year." ------------------------------------------------------------------------ Then function returns incorrect html string (sorry, i cant show it here). And then we give this incorrect string to html2text function, then this function will not return and procedure will halt... At the moment i have removed html2normal function from some problematic places in function parsehttpstream It's best, then you subscribe to scrintingnews rss... Greetings Toomas |
|
From: Toomas Toots <tom@i-...> - 2003-04-25 10:51:20
|
Hi! In one email Marcus wrotes: 5. we should change the internal data-handling to use objects and collections instead of records and arrays, that makes a lot of things easier and reduces unnecessary copying of data and removes some memory-leaks that are present in the code What can we do with this problem? Is it ig job to change data structures to something more normal than records and arrays? At the moment, i think, is quite uneconomical to load xml data to array first and then from array to treeview data records. Greetings Toomas |
|
From: Marcus Hettlage <hmarcus@us...> - 2003-04-24 11:39:59
|
Hi! I've used your fixes for a long time and, congratulations they work. I've a small additional change. Some feeds are composed of multiple source feeds into a single feed. These feeds also provide the origin of the headline. I've added support for this in the form of a small header that is displayed for every headline. I'll check this in and make a build. If I hear no further comments I'll make a new release at 16:00 GMT+1. Greetings Marcus ----- Original Message ----- From: "Toomas Toots" <tom@...> To: "Feedreader Development" <feedreader-development@...> Sent: Thursday, April 24, 2003 12:52 PM Subject: [Feedreader-development] New release? > Hi! > > I have made some (a lot of?) changes to feedreader. You can get it from > CVS. Is it time to release new build? Please test it and comment it. > > I do not know, how to release new builds in sourceforge. Can Marcus, the > Buildmaster, do it? ;) > > > Greetings > Toomas > > > > ------------------------------------------------------- > This sf.net email is sponsored by:ThinkGeek > Welcome to geek heaven. > http://thinkgeek.com/sf > _______________________________________________ > Feedreader-development mailing list > Feedreader-development@... > https://lists.sourceforge.net/lists/listinfo/feedreader-development |
|
From: Toomas Toots <tom@i-...> - 2003-04-24 10:53:06
|
Hi! I have made some (a lot of?) changes to feedreader. You can get it from CVS. Is it time to release new build? Please test it and comment it. I do not know, how to release new builds in sourceforge. Can Marcus, the Buildmaster, do it? ;) Greetings Toomas |
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-04-21 21:00:20
|
Slashdotted in context of RSS and RSS readers, but still a noteworthy = link: http://books.slashdot.org/books/03/04/16/1438224.shtml?tid=3D192&tid=3D15= 6 Kristjan |
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-04-16 16:51:37
|
-- Add new feed --
[Cancel] and [Next] button using non-standard placement and naming =
conventions:
- on first page:
[ Cancel ][ Next ]
- on second page
[ Previous ][ Finish ]
All standard wizard dialogs (InstallShields etc) use
[ < Back ][ Next > ] [ Cancel ]
and
[ < Back ][ Finish ] [ Cancel ]
-- Right pane --
- In rightclick menu there's "Delete current feed". A messagebox =
afterwards asks "Do you want to delete active feed?" Is "current" =3D =
"active"? I suggest to use "current" word only.
- when updating feedlist, "New feeds" window is nice, but I cant do =
anything on this window, clicking does not help. May just make make new =
feeds in feedlist _bold_? Or use a "new" icon?
-- About dialog --
- About dialog is too big for such a small application (most of the time =
I keep FR main window small, so About box is larger than Feedreader =
itself :)
- Adding tabs is a neat idea, but I'd get rid of the title graphic =
(until we got a cool image to put there - a la Photoshop logo :)
- Should we stress a GPL licence when we have a special tab for this?
My concept sketch for About dialog:
http://www.feedreader.com/node.php?id=3D262
-- Taskbar popup --
- It displays items in reverse ordrer. The correct order should be the =
same as in the item's window (newest in top)
-- Exit dialog --
- "...is closing up" is using too large font. I suggest to use standard =
system font size.
Cheers,
Kristjan
|
|
From: Toomas Toots <tom@i-...> - 2003-04-02 13:57:32
|
If Feedreader is started minimized, then it will not show any newmessage
windows until you open form first time.
Problem is in procedure:
procedure TFeedThread.OnNewHeadlinesAvailable;
...
For i := 0 To gMyFeedsTreeNode.Count-1 Do Begin
TmpNode := gMyFeedsTreeNode.Item[i];
Info := feedinfo( TmpNode.Data );
If Info = Nil Then Continue;
If Info.feedid = WorkerThread.CurFeedID Then Begin
Node := TmpNode; <-------it will never reach this place, if
application is started minimized and form is not showed.
Break;
End;
End;
If Node = Nil Then <--------------------
Exit;
...
Every time Node will be nil and procedure will be terminated. If I open
mainform, then node will not be nil any more and all newmessage windows
will be showed.
Diference beetween minimized start and normal start:
//////////////////////////////
if gProperties.StartHidden Or (LowerCase(ParamStr(1)) = 'min') or
(LowerCase(ParamStr(2)) = 'min') then Begin
Application.ShowMainForm := False;
Self.windowstate := wsminimized;
fShowWindowInTaskbar := False;
End
Else fShowWindowInTaskbar := True;
If fShowWindowInTaskbar Then Begin
ShowWindow( Application.Handle, SW_SHOWNORMAL );
Application.ShowMainForm := True;
End;
//////////////////////////////////////
Second problem is about animated tray icon. It acts kind of funny. Try
to test it with "enable animated trayicon" setting allowed. It will go
red even if there is no new messages if some items are unread.
////////////////////////////////////////////////////////////////////////////////////////////
Some problems here....
Procedure TMainWindow.AnimateTrayIcon( Enable: Boolean );
Begin
If Enable Then Begin
If gProperties.EnableTaskbarAnimations Then Begin
If Not CoolTrayIcon1.CycleIcons Then Begin
CoolTrayIcon1.IconList := TrayIconAnimatedImageList;
CoolTrayIcon1.CycleIcons := True;
End;
End
Else Enable := False;
End;
If Not Enable Then Begin
CoolTrayIcon1.CycleIcons := False;
CoolTrayIcon1.IconList := TrayIconStaticImageList;
If gfUnseenNewsAvailable Then
CoolTrayIcon1.IconIndex := 1
Else
CoolTrayIcon1.IconIndex := 0;
End;
End;
I'm trying to fix them but if anybody sees problem right now, replay
please...
|
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-04-01 11:26:47
|
Some final remarks: --- Seach headlines --- - Search dialog should be moved from the left pane to the top-right = pane. This gives an user better visual tie between the search form and = the results. - "Cancel" button is uneccesary - It also allows to add "Advanced search options" pane in later phase of = delopment. An example: http://www.feedreader.com/node.php?id=3D261 (My mockup could be a little bit cramped space-wise. Also, search field = might be a bit longer) I also updated the "Subscribe" mock-up to be consistent with search = dialog: http://www.feedreader.com/node.php?id=3D260 Kristjan |
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-03-31 23:07:55
|
> Btw, no second page of wizard ("Please enter the name of the=20
> feed") the [XML] icon is mysteriously gone already :)
Should be read "Btw, on second page of wizard..."=20
> - Ideally these fields should be on the second step of=20
> wizard, but I understand this is not possible to implement -=20
> FR must connect and validate feed (using loging/password info=20
> if neccessary) before moving from step 1 to step 2, right?
I'd still like to see username/password on step 2:
- On step 1 FR tries to connect as usual
- when returning a HTTP 403 (or 401?), it creates a extra a step 1A, =
asking
for username/password - "This feed requires valid username and =
password"...
- When these provided, FR continues to usual step 2
> I suggest to introduce a narrow information bar above the=20
> headline pane, with a text like "You can subscribe this feed=20
> by clicking here: [subscribe]". The bar should appear just=20
> like "Search" pane, sliding from the top. =20
I made a quick mockup:
http://www.feedreader.com/node.php?id=3D260
Anybody has better ideas?
> In next step we can start discussing better=20
> structure/arrangement of the properties.
Here's my initial suggestion.
I kept all the options the same, just moved them around to
be more logical and consistent.=20
On the left is the tree hierarchy, on the right are properties.=20
I comment each section separately.
_____________________________________
Startup [] Run program on startup
[] Start minimized to tray
[] Check msgs on startup
- What are "msgs"? Are we using such a term=20
elsewhere? Are we "checking" or "updating"?=20
Should be replaced with "Update feeds on startup"
_____________________________________
Appearance [] Show time
[] Show date
[] Show headlines with topics
[] Show all article tags
[] Show newsfeed images if available
[] Show tree lines=20
[] Show Feed-Hints
=09
Font:
[Select font]
[] Open links in new browser window
- This is the longest properties page,
preferrably should be grouped into smaller sets.
I separated different functional groups with an empty line.
- Show time and date *where*?=20
Are these columns in feed item's pane?
- "Feed-Hints" -> "Feed hints"=20
- What are "tree lines" and "Feed-Hints" anyway?
_____________________________________
Notification [] Show pop-up-window in taskbar
box Visibility duration (sec) [ |v]
[] Use notification box fade in and out
[] Enable taskbar animations
- Is it "notification box" or "pop-up-window"? I prefer
the former. Unify.
- I am not sure about the "taskbar animations" option belonging here.=20
Whereelse?
_____________________________________
Connection [] Use proxy server
Address : _____ Port : ___
Username : _____ =20
[Get settings from the Internet Explorer]
[x] Get proxy settings from IE on startup
- Button should be read:
"Get proxy settings from the Internet Explorer"
- I need some kind of feedback message when my=20
IE proxy settings are *not* set. Currently I click=20
on the button and nothing happens. I is leaving me
clueless.=20
_____________________________________
Webserver [] Enable local Userland-Sever
[] Enable local AmphetaDesk-webserver
[] Enable internal WebServer=20
Port : ___
- A explaing message what the webserver is good for
would be a godsend.
- Note different naming conventions for words "local/internal"=20
and "WebServer". Must be unified.
_____________________________________
??? Feed defaults
Refresh: _____
Archive:_____=20
[] Play sounds
[] Show XMLParser errors
[] Enable javascript
- This section can also be called: "Leftovers" or=20
"I Dont Have an Idea Where to Put Those" :)
- What sounds? When? On incoming messages? Explain!
- "XMLParser" -> "XML parser"
- "javascript" -> "Javascript"
_____________________________________
One more general note: there is no consistent=20
spacing conventions on dialogs. Sometimes FR uses
Refresh : [ |v]
and sometimes
Refresh: [ |v]
(note the missing space)
Kristjan
|
|
From: Kristjan Jansen <kristjan.jansen@sl...> - 2003-03-31 20:03:25
|
MAIN WINDOW=20 --- "New..." button --- 1) Maybe it's just me, but I feel most applications have "New" button = left side of the toolbar. To be consistent I'd suggest to swap the "New" = and "Update" buttons. Up for discussion. 2) "New..." button dropdown menu is compete mess.=20 - What on earth is "External" and "Local" feeds? Ok, I know, I chatted = with Toomas and "Local" feeds seems to be a work-in-progress to create = outcoming (exportable) feeds. Can I use such a functionality? No, at = least not at a first glance - the internal feed stays empty, I'm not = explained how to create new items there. Solution: Remove the "Local" menu opion until the actual functionality = is there. Just keep two options "New feed" and "New folder". - Why the "New..." menu is auto-hiding? = http://www.feedreader.com/node.php?id=3D259 (like Word and other = Microsoft applications, where not-so-often-used menu options are hidden = from users). C'mon - there's only 3 options, auto-hiding them is really = annoying. - Will "Add new feedlist" be back in the menu?=20 --- Seach headlines --- Dialog color scheme is weird. Shouldn't the text entry field be white = (not gray) to be consistent with the rest of the application? --- Feedlist (the leftmost pane) --- 1) The rightbutton menu usage should be reconsidered: - Why there's no rightbutton menu on "My feeds"?=20 - Why folder-creating is available under the toolbar, not in the = feedlist itself (like standard Windows explorer tree)? - Why I can't add or delete feedlists? 2) The name "Estonians channels" are misleading. Are we using the term = "channel" elsewhere? No? Be consistent, use the term "feed" thoroughly.=20 Solution: rename it to "Estonian feeds"=20 3) Favourites. Ok, I see these are my IE favourites, but what the hell they are doing = here, in Feedreader?=20 -- Feed items (the upper-right pane) --- "Date" and "Headline" column headers are clickable, but clicking on them = does nothing. I'd expect clicking will change the ordering and sorting = of items (like Outlook). Solutions: - allow ordering and sorting - alternatively, just remove clickability --- Statusbar --- Why there's two progressbars, one continious, one stepped? Can't we = settle with one? DIALOGS --- Add new feed --- 1) There's currently two titles on the dialog: "New feed wizard" + "[XML] Adding new feed" IHMO such a duplication is unneccessary. We do not have to emphasize = that we are using "wizard" here.=20 Solution: - Just use a string "Add new feed" in titlebar, remove the second = "Adding new feed" line.=20 - Also remove XML icon, it's usage here is incorrect. We need a proper = icon/graphic here.=20 Btw, no second page of wizard ("Please enter the name of the feed") the = [XML] icon is mysteriously gone already :) 2) "Username" and "Password" are too intrusive in first step of the = wizard. They seem to FORCE user to fill these fields - there's no = information are these fields optional or not (In fact, the whole dialog = looks like a standard login box, more or less). Solutions: - Write short explanation what these fields are and stress that they are = OPTIONAL.=20 - One should also consider to make the dialog expand/collapse in the = bottom (like the Windows login box) - something like "v Advanced = options". When expanded, power user can specify login/password, when = collapsed, novice user just won't see the fields.=20 - Ideally these fields should be on the second step of wizard, but I = understand this is not possible to implement - FR must connect and = validate feed (using loging/password info if neccessary) before moving = from step 1 to step 2, right? 3) Change "Next" to "Next >" and "Previous" to "< Previous" See also http://ranchero.com/images/nnw/subscribe1.jpg and http://www.feedreader.com/node.php?id=3D253 --- Subscribe from "Available feeds" --- The "Subscribe" button what appears on toolbar is almost unnoticable for = user.=20 I suggest to introduce a narrow information bar above the headline pane, = with a text like "You can subscribe this feed by clicking here: = [subscribe]". The bar should appear just like "Search" pane, sliding = from the top. =20 --- Feed properties --- - Once again, I am against the [XML] image. Yep, I know the dialog badly = lacks color, but the [XML] is meant to be the link, and should have a = tooltip when hovering above the icon. Using it here as a static = illustation is misguiding. Solution: create a new fancy icon :) - Again, IHMO "Username" and "Password" should not be emphasized - they = should be moved to the bottom, separated with a horizontal line (with a = be a short explaination?) - BTW, you can put these fields on a same line: Username: [ ] Password: [ ] - "Directly show headline link" Whatthehellizit? I'm clueless. - "Archive" and "Refresh feed" show "Default" as a standard option. Why = not display the *actual default value* instead?=20 If one really wants to stress that we are dealing with defaults (why?), = he could write "50 headlines - default" or similar. When I am new to FR, I might not even touched the global properties and = never changed defaults, so I never know what the actual value is. - Left side of the "Refresh feed" field must be aligned with other = fields, right now it is shorter than others. --- (Global) Properties --- There's currently two methods of sturcturing - tabs and checkbox-list. = Why not settle for classic tree+properties pane, like IE, Mozilla and = numerious other applications? See NewzCrawler preferences windows: = http://www.feedreader.com/module.php?mod=3Dimage&uid=3D2 (of course it = is bloated, using too much structuring, but you will get the idea) Such way options can still be nested/structured, but properties dialog = itself is far more logical and familiar to user.=20 Current checkbox-list does not provide enough flexibility. What if I = want to add a different kind of field to "General > Misc > Play sounds", = say a button "Select a sound sample"? I'd start with following: - replace current checbox-list component with standard tree+properties = component - create properties pages for "Startup" and "Miscallenous" (not "Misc"! = -=20 do not use abbreviations) - merge the "Appearance" checbox-list and "Appearance" tab into a single = "Appearance" properties page. - move "Connection", and "Webserver" properties under the general tree, = get rid of tabs. In next step we can start discussing better structure/arrangement of the = properties. -- About Feedreader -- - "Version" and "Build" number fields look like a text-entry fields. = Can't they be outputted as a plain text, not as a form control? - Check for new version work ok, but leaves the user clueless. Right, = new version is avaliable...now what? The dialog should give me some = clues how to proceed, at least provide a link to http://www.feedreader.com page = (even better, a downloads page) More to come :) Regards, Kristjan |
|
From: Marcus Hettlage <hmarcus@us...> - 2003-03-31 10:55:56
|
I haven't yet noticed any problems with starting the app minimized. But I've found and fixed a problem with the periodic refresh. If you at least once refresh a feed manually the automatic update of all feeds is stopped forever. I'll upload the fixes today, after some additional testing. Greetings Marcus ----- Original Message ----- From: "Toomas Toots" <tom@...> To: "Feedreader Development" <feedreader-development@...> Sent: Monday, March 31, 2003 12:37 PM Subject: [Feedreader-development] Some problems, then testing CVS version > I have found two problems, then using CVS version of Feedreader. > > 1. If Feedreader starts minimized to tray, then it will not show any > "new item notification", until i open it from tray. And if i'm not > opening it from tray, Feedreader can run whole day and not perform any > updates. > > 2. Feedreader somehow forgets to update subscribed feeds. It does not > happen very often, but for example once in a day. I think it's because > of some incorrect feed but it should not stop updating because of > incorrect feeds. > > > I will dig into code myself and try to fix these bugs but if someone > (Marcus? ;)) knows about these bugs, he can gove me some hints or fix this. > > > Greetings > Toomas > |
|
From: Toomas Toots <tom@i-...> - 2003-03-31 10:37:55
|
I have found two problems, then using CVS version of Feedreader. 1. If Feedreader starts minimized to tray, then it will not show any "new item notification", until i open it from tray. And if i'm not opening it from tray, Feedreader can run whole day and not perform any updates. 2. Feedreader somehow forgets to update subscribed feeds. It does not happen very often, but for example once in a day. I think it's because of some incorrect feed but it should not stop updating because of incorrect feeds. I will dig into code myself and try to fix these bugs but if someone (Marcus? ;)) knows about these bugs, he can gove me some hints or fix this. Greetings Toomas |
|
From: Miha Remec <pb0301@mi...> - 2003-03-19 23:12:07
|
Comment on build 599: Toolbar settings are not saved when app receives WM_QUERYENDSESSION message. Line actionmanager1.SaveToFile( gProperties.DataDirectory + '\Toolbarsettings.bin'); should be moved from TMainWindow.FormClose to TMainWindow.PrepareAppTermination procedure. Also, TMainWindow.FormClose contains some other assignment statements that probably should be moved to TMainWindow.PrepareAppTermination procedure. -- Miha Remec |
|
From: Kyle Cordes <kyle@ky...> - 2003-03-18 14:25:54
|
From: "Toomas Toots" <tom@...> > And what about bugs that Kyle wrote about: > > 1) Feedreader sometimes write invalid XML to one of its data files > > 2) when reading the file back in, it fail to parse, then fail silently with > no error message, yet lose the "read / not read" status of every item in > every feed. (Code which fails silently in there is an error, is an > extremely bad idea, by the way.) I will hopefully find time soon to send an example of a feed that triggers this problem. However, the very existance of code that will fail silently, is a large problem. That code needs to fixed, regardless of what specific example makes it happen. Kyle |
|
From: Toomas Toots <tom@i-...> - 2003-03-18 11:58:25
|
I guess, it's quite new thing. Take a look at http://sourceforge.net/export/rss2_project.php?group_id=70179 Maybe we should add some of these feeds (at least "project file releases") to Feedreader "My feeds" section, then user installs it. So users can get notified about new releases of Feedreader. Greetings Toomas |
|
From: Marcus Hettlage <hmarcus@us...> - 2003-03-18 08:50:16
|
Currently I can't reproduce the problems Kyle described.
I'd like to fix it but I've no idea where it could happen.
As I said I don't have the problem.
Toomas ::
I have compiled and tested CVS version. Before new release i'd like to =
commit some features i wrote about in previous email. I guess I can do =
it today.=20
I think that there is some little problem with automatic updating =
engine. I notice that if i let FR to run for example one day it can stop =
updating feeds automatically. Only program restart helps. Even if I push =
update button, it does not trigger update.
And what about bugs that Kyle wrote about:
1) Feedreader sometimes write invalid XML to one of its data files
2) when reading the file back in, it fail to parse, then fail silently =
with
no error message, yet lose the "read / not read" status of every item in
every feed. (Code which fails silently in there is an error, is an
extremely bad idea, by the way.)
Greetings
Toomas
Marcus Hettlage ::
What about a new release?
I think it's time for it. I've fixed some
bugs and we've got new features that
users requested.
Has someone tested the current source-version,
except myself?
Greetings
Marcus
|
|
From: Marcus Hettlage <hmarcus@us...> - 2003-03-18 08:47:51
|
Ok, both comments are reflected in the code.
> Comments about build 594:
>
> 1) There's still a bug related to http proxy settings in
> AddOptionsDialog_FRM.pas, line 393:
>
> If gProperties.ProxyEnabled Then
> if gProperties.ProxyPassword = '' then
> if gProperties.ProxyUsername <> '' Then
> gProperties.ProxyPassword := InputBox( TranslateString('Enter
proxy
> server password'), TranslateString('for user') + ' ' +
> gProperties.ProxyUsername, '');
>
> gProperties.ProxyEnabled := enableproxy.checked;
> gProperties.ProxyHost := proxyhost.text;
> gProperties.ProxyPort := proxyport.text;
> gProperties.ProxyUsername := proxyusername.text;
>
> As you can see, password dialog window will open even when enableproxy
> checkbox will not be checked because gProperties.ProxyEnabled are checked
> first. You should put those four assignment statement before checking for
> password prompt.
>
> 2) I prefer that minimizing window would not occur on close event
> (MainForm_FRM.pas, TMainWindow.FormCloseQuery). IMO, it is necessary to
> minimize main window before program closing program.
>
> Here's a patch that will not minimize window if program is closing.
>
> Index: MainForm_FRM.pas
> ===================================================================
> RCS file: /cvsroot/feedreader/feedreader/MainForm_FRM.pas,v
> retrieving revision 1.17
> diff -r1.17 MainForm_FRM.pas
> 841c841
> < if not fProgrammaticApplicationClose then
> ---
> > if not fProgrammaticApplicationClose then begin
> 843c843,844
> < application.minimize;
> ---
> > application.minimize;
> > end;
>
> or, if you prefer whole code:
>
> procedure TMainWindow.FormCloseQuery(Sender: TObject; var CanClose:
> Boolean);
> begin
> if not fProgrammaticApplicationClose then begin
> canclose := false;
> application.minimize;
> end;
> end;
>
>
> --
> Miha Remec
>
>
>
>
> -------------------------------------------------------
> This SF.net email is sponsored by:Crypto Challenge is now open!
> Get cracking and register here for some mind boggling fun and
> the chance of winning an Apple iPod:
> http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en
> _______________________________________________
> Feedreader-development mailing list
> Feedreader-development@...
> https://lists.sourceforge.net/lists/listinfo/feedreader-development
|
|
From: Toomas Toots <tom@i-...> - 2003-03-18 08:14:15
|
I have compiled and tested CVS version. Before new release i'd like to commit some features i wrote about in previous email. I guess I can do it today. I think that there is some little problem with automatic updating engine. I notice that if i let FR to run for example one day it can stop updating feeds automatically. Only program restart helps. Even if I push update button, it does not trigger update. And what about bugs that Kyle wrote about: 1) Feedreader sometimes write invalid XML to one of its data files 2) when reading the file back in, it fail to parse, then fail silently with no error message, yet lose the "read / not read" status of every item in every feed. (Code which fails silently in there is an error, is an extremely bad idea, by the way.) Greetings Toomas Marcus Hettlage :: > What about a new release? > I think it's time for it. I've fixed some > bugs and we've got new features that > users requested. > Has someone tested the current source-version, > except myself? > > Greetings > Marcus |
|
From: Miha Remec <pb0301@mi...> - 2003-03-18 07:17:57
|
Comments about build 594:
1) There's still a bug related to http proxy settings in
AddOptionsDialog_FRM.pas, line 393:
If gProperties.ProxyEnabled Then
if gProperties.ProxyPassword = '' then
if gProperties.ProxyUsername <> '' Then
gProperties.ProxyPassword := InputBox( TranslateString('Enter proxy
server password'), TranslateString('for user') + ' ' +
gProperties.ProxyUsername, '');
gProperties.ProxyEnabled := enableproxy.checked;
gProperties.ProxyHost := proxyhost.text;
gProperties.ProxyPort := proxyport.text;
gProperties.ProxyUsername := proxyusername.text;
As you can see, password dialog window will open even when enableproxy
checkbox will not be checked because gProperties.ProxyEnabled are checked
first. You should put those four assignment statement before checking for
password prompt.
2) I prefer that minimizing window would not occur on close event
(MainForm_FRM.pas, TMainWindow.FormCloseQuery). IMO, it is necessary to
minimize main window before program closing program.
Here's a patch that will not minimize window if program is closing.
Index: MainForm_FRM.pas
===================================================================
RCS file: /cvsroot/feedreader/feedreader/MainForm_FRM.pas,v
retrieving revision 1.17
diff -r1.17 MainForm_FRM.pas
841c841
< if not fProgrammaticApplicationClose then
---
> if not fProgrammaticApplicationClose then begin
843c843,844
< application.minimize;
---
> application.minimize;
> end;
or, if you prefer whole code:
procedure TMainWindow.FormCloseQuery(Sender: TObject; var CanClose:
Boolean);
begin
if not fProgrammaticApplicationClose then begin
canclose := false;
application.minimize;
end;
end;
--
Miha Remec
|
|
From: Marcus Hettlage <hmarcus@us...> - 2003-03-18 01:50:12
|
What about a new release? I think it's time for it. I've fixed some bugs and we've got new features that users requested. Has someone tested the current source-version, except myself? Greetings Marcus |
