Bug 167515

Summary:	support for comic titles and title text
Product:	[Unmaintained] plasma4	Reporter:	a thing <a_thing>
Component:	widget-comic	Assignee:	Tobias Koenig <tokoe>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	mat69
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	adds the title for xkcd adds support for strip titles in the comic app

Description a thing 2008-07-27 09:20:45 UTC

Version:            (using KDE 4.0.98)
Installed from:    Fedora RPMs

Currently the comic plasmoid only displays a picture. However, for xkcd (and maybe other comics), the title of the comic and the title text (the XHTML title attribute for the img tag that is displayed on a mouseover in graphical browsers) are often a big part of the comic and the plasmoid does not display these.

Comment 1 Dominik Tritscher 2008-07-28 17:41:35 UTC

Changed to a wishlist item, as it's not a bug.

Comment 2 Matthias Fuchs 2008-08-25 21:03:50 UTC

Created attachment 27041 [details]
adds the title for xkcd

Ok, I created a patch that adds a virtual QString stripTitle() method to comicprovider and cachedprovider, the title is also stored. I also modifed xkcdprovider to provide the title.

Here is the patch for the dataengine.

Comment 3 Matthias Fuchs 2008-08-25 21:05:19 UTC

Created attachment 27043 [details]
adds support for strip titles in the comic app

Additionally I modified the applet to show the title of the comic on top.

I'm not sure though if I should provide an option to deactivate that in the config, or if I should also show the comic title --> "xkcd - Voting Machines" or "xkcd" if there is no strip. Currently it is only the strip title.