Summary: | Option per channel to load the webpage directly | ||
---|---|---|---|
Product: | [Applications] akregator | Reporter: | Stian Haklev <shaklev> |
Component: | general | Assignee: | kdepim bugs <kdepim-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | malefizer, phoenixreads |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Debian testing | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Stian Haklev
2004-06-01 21:07:34 UTC
Could you please explain that in more detail? I feel dumb today. I think he meant (at least that's what I mean, and why I gave 5 votes for it), that most rss feeds only show a one-line (it at all) description of the content of the article, and you have to click "full story" in the kthml-view on the bottom right. Therefore you should be able to say (on a per-channel basis), that the article should be automatically loaded in the khml-frame at the bottom right instead of the useless header and the "full story" link. This would save you from the need to click "full story" to open the article in a new tab (or to middleclick the story in the article browser). Hopefully it's clear now, I am not a native English-speaker either. Otherwise please ask again, but this is a very simple but powerfull feature. You also wouldn't want to click "full story" in your e-mail client either. Okay understood. *** Bug 85531 has been marked as a duplicate of this bug. *** I implemented this tonight to open a new tab when you press the item, but I don't think that's good idea. And I don't think we should display html pages in the articleviewer's small box, so that's not a choice. I prefer using middle mouse button press on the item to open it in background tab or anything you want, but please elaborate more how this could be made usable. For me this looks like a mess, when you can't know does clicking the item open a tab or to display the content of the feed in the "small box". But please, give me some comments about this.. PS. Developers, please don't reassign bugs to yourself even if you're working on it. But if you reassign, please add akregator-devel@lists.sourceforge.net to the CC list so every developer will be noticed when there's a new comment. We've discussed about this on the irc channel and we decided not to implement this because of usability reasons. *** Bug 92512 has been marked as a duplicate of this bug. *** I fail to understand why there are usability issues with this. It would just be konqueror on 50% of the screen 'articleviewer's small box'? Where is it small? It is pretty big here on 1024x768. Other than that, it's just *an option*, so it doesn't hurt. The articleviewer is totally useless for some channels and I'd rather not see the channels at all. Improvement to this idea: * use a different stylesheet (as many websites (blogs) provide alternate stylesheets (for printing, as an example)) probably a little bit altered (like hide pictures, change font size etc. - KHTML is versatile, as we all know it) Actually this seems to be nice addition, got it wrong before.. :) SVN commit 410054 by rytilahti: Open the webpage directly to articleviewer if there's no description available. This will be here for testing and if it's found out cool then we could add an option and prolly some cleaning (use media/print stylesheet if available and such) BUGS:82660 M +4 -1 trunk/KDE/kdepim/akregator/src/articleviewer.cpp --- trunk/KDE/kdepim/akregator/src/articleviewer.cpp #410053:410054 @@ -404,7 +404,10 @@ m_viewMode = NormalView; disconnectFromNode(m_node); m_node = 0; - renderContent( formatArticle(article.feed(), article) ); + if(article.description().isEmpty()) + openURL(article.link()); + else + renderContent( formatArticle(article.feed(), article) ); } void ArticleViewer::slotSetFilter(const ArticleFilter& textFilter, const ArticleFilter& statusFilter) IMHO, this should be an option but not the default behavior. "article.description().isEmpty()" is not enough to mark an article's content usefull. It should be an option since this doesn't cover all the cases when the content is only a bit more than nothing. Thanks for reconsidering. SVN commit 412913 by osterfeld: make "show linked website" configurable per feed and disable it by default CCBUG: 82660 M +1 -1 trunk/KDE/kdepim/akregator/src/articleviewer.cpp M +15 -0 trunk/KDE/kdepim/akregator/src/feed.cpp M +5 -0 trunk/KDE/kdepim/akregator/src/feed.h M +13 -1 trunk/KDE/kdepim/akregator/src/propertiesdialog.cpp M +2 -0 trunk/KDE/kdepim/akregator/src/propertiesdialog.h M +26 -18 trunk/KDE/kdepim/akregator/src/propertieswidgetbase.ui |