Merge lp:~eduardo-mucelli/cairo-dock-plug-ins-extras/WebSearch into lp:~cairo-dock-third-party/cairo-dock-plug-ins-extras/third-party

Proposed by Eduardo Mucelli Rezende Oliveira
Status: Merged
Merged at revision: 383
Proposed branch: lp:~eduardo-mucelli/cairo-dock-plug-ins-extras/WebSearch
Merge into: lp:~cairo-dock-third-party/cairo-dock-plug-ins-extras/third-party
Diff against target: 104 lines (+12/-5)
4 files modified
WebSearch/Changelog.txt (+1/-0)
WebSearch/WebSearch (+7/-3)
WebSearch/WebSearch.conf (+3/-1)
WebSearch/auto-load.conf (+1/-1)
To merge this branch: bzr merge lp:~eduardo-mucelli/cairo-dock-plug-ins-extras/WebSearch
Reviewer Review Type Date Requested Status
Matthieu Baerts Approve
Review via email: mp+179545@code.launchpad.net

Description of the change

WebSearch: List of results now shows the search engine's logo if is not a "thumbnailable" link icon. As requested on the CD Forum.

To post a comment you must log in.
Revision history for this message
Eduardo Mucelli Rezende Oliveira (eduardo-mucelli) wrote :

Mattbe, the lastmodif was not modified this time, so I did not update the list.conf for WebSearch.

Revision history for this message
Matthieu Baerts (matttbe) wrote :

> the lastmodif was not modified this time, so I did not update the list.conf for WebSearch.

You don't need to read/modify this file because it's only managed by the dock ;)

But you can update 'list.conf' file because this file will be used by the dock (which will download it from our server) in order to know if applets have to be updated. (the dock will regularly download this file from our server and it will compare the number available in this 'last-modif' file and the number linked to the 'last modif' key in this 'list.conf' file). (I hope it's understandable, I'm a bit tired ^^)

In other words, each time you made a modification, it would be great to modify the 'last modif' key by the current date in this 'list.conf' file :) (e.g. check the rev 381 in lp:cairo-dock-plug-ins-extras ;) )

PS: oh sorry, I didn't see that these modifications (your rev 381) were not in the trunk, I'm going to merge it asap ;)

review: Approve
Revision history for this message
Eduardo Mucelli Rezende Oliveira (eduardo-mucelli) wrote :

Ah, got it. Thanks matttbe!

Eduardo Mucelli R. Oliveira
On Aug 10, 2013 1:10 AM, "Matthieu Baerts" <email address hidden> wrote:

> Review: Approve
>
> > the lastmodif was not modified this time, so I did not update the
> list.conf for WebSearch.
>
> You don't need to read/modify this file because it's only managed by the
> dock ;)
>
> But you can update 'list.conf' file because this file will be used by the
> dock (which will download it from our server) in order to know if applets
> have to be updated. (the dock will regularly download this file from our
> server and it will compare the number available in this 'last-modif' file
> and the number linked to the 'last modif' key in this 'list.conf' file). (I
> hope it's understandable, I'm a bit tired ^^)
>
> In other words, each time you made a modification, it would be great to
> modify the 'last modif' key by the current date in this 'list.conf' file :)
> (e.g. check the rev 381 in lp:cairo-dock-plug-ins-extras ;) )
>
>
> PS: oh sorry, I didn't see that these modifications (your rev 381) were
> not in the trunk, I'm going to merge it asap ;)
> --
>
> https://code.launchpad.net/~eduardo-mucelli/cairo-dock-plug-ins-extras/WebSearch/+merge/179545
> You are the owner of
> lp:~eduardo-mucelli/cairo-dock-plug-ins-extras/WebSearch.
>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'WebSearch/Changelog.txt'
2--- WebSearch/Changelog.txt 2013-06-16 00:59:02 +0000
3+++ WebSearch/Changelog.txt 2013-08-09 21:39:04 +0000
4@@ -1,3 +1,4 @@
5+1.4.8: (August/9/2013): List of results now shows the search engine's logo if is not a "thumbnailable" link icon.
6 1.4.7: (June/15/2013): Fixed Flickr.
7 1.4.6: (June/14/2013): Fixing Google and Bing search. Removed Webshots and Twitter. Webshots closed :( its doors and Twitter changed its search to something really hard to fetch.
8 1.4.3: (January/31/2012): Fixing Youtube search. Fixing a problem when opening URLs.
9
10=== modified file 'WebSearch/WebSearch'
11--- WebSearch/WebSearch 2012-02-01 01:57:11 +0000
12+++ WebSearch/WebSearch 2013-08-09 21:39:04 +0000
13@@ -16,7 +16,7 @@
14 # GNU General Public License for more details.
15
16 # This applet provides an interface to some search engines such as
17-# Google, Bing, Teoma, Yahoo!, Youtube, Webshots, Flickr, Wikipedia, ImageShack, and Twitter.
18+# Google, Bing, Teoma, Yahoo!, Youtube, Flickr, Wikipedia, and ImageShack.
19 # To choose the search engine you can
20 # (1) Right-click on the main icon -> WebSearch -> (Choose the engine)
21 # (2) Right-click -> Configure this applet -> Configuration -> Search engine
22@@ -80,7 +80,7 @@
23 attr_accessor :engine, :engines, :query, :history,
24 :number_of_fetched_links, :number_of_displayed_links, :page_of_displayed_links,
25 :show_current_page, :show_description_instead_url, :show_thumbnail_preview,
26- :scroll_engine_index
27+ :scroll_engine_index, :show_logo_result
28
29 DialogActiveTime = 5 # time in seconds that the dialog window will be active
30
31@@ -106,6 +106,7 @@
32 self.show_current_page = conf.params['Configuration']['show current page'].to_b
33 self.show_description_instead_url = conf.params['Configuration']['show description instead url'].to_b
34 self.show_thumbnail_preview = conf.params['Configuration']['show thumbnail preview'].to_b
35+ self.show_logo_result = conf.params['Configuration']['show logo result'].to_b
36 end
37
38 def start
39@@ -162,7 +163,7 @@
40 item['label'] = engine
41 item['menu'] = 1
42 item['id'] = i
43- item['icon'] = File.expand_path("./images/data/#{engine}.png")
44+ item['icon'] = engine.icon
45 items.push item
46 end
47 items
48@@ -345,6 +346,9 @@
49 else
50 sub_icon_list << link.shortened_url # user prefer see shortened url with the sub-icon
51 end
52+ if not link.instance_of?(ThumbnailedLink) and self.show_logo_result
53+ link.icon = File.expand_path("./images/data/#{self.engine.name}.png")
54+ end
55 sub_icon_list << link.icon # the icon
56 sub_icon_list << link.id.to_s # the sequential id
57 end
58
59=== modified file 'WebSearch/WebSearch.conf'
60--- WebSearch/WebSearch.conf 2013-06-16 00:59:02 +0000
61+++ WebSearch/WebSearch.conf 2013-08-09 21:39:04 +0000
62@@ -1,4 +1,4 @@
63-#1.4.7
64+#1.4.8
65
66 #[gtk-about]
67 [Icon]
68@@ -109,3 +109,5 @@
69 #b Enable thumbnail preview for Youtube, Flickr, ImageShack, and Digg searches ?
70 #{for slow connections, disable will result in significantly faster fetching.}
71 show thumbnail preview = true
72+#b Show search engine's logo as the icon on the results?
73+show logo result = true
74
75=== modified file 'WebSearch/auto-load.conf'
76--- WebSearch/auto-load.conf 2013-06-16 00:59:02 +0000
77+++ WebSearch/auto-load.conf 2013-08-09 21:39:04 +0000
78@@ -10,4 +10,4 @@
79 category = 3
80
81 # Version of the applet; change it everytime you change something in the config file. Don't forget to update the version both in this file and in the config file.
82-version = 1.4.7
83+version = 1.4.8
84
85=== modified file 'WebSearch/images/data/Bing.png'
86Binary files WebSearch/images/data/Bing.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Bing.png 2013-08-09 21:39:04 +0000 differ
87=== modified file 'WebSearch/images/data/Digg.png'
88Binary files WebSearch/images/data/Digg.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Digg.png 2013-08-09 21:39:04 +0000 differ
89=== modified file 'WebSearch/images/data/Flickr.png'
90Binary files WebSearch/images/data/Flickr.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Flickr.png 2013-08-09 21:39:04 +0000 differ
91=== modified file 'WebSearch/images/data/Google.png'
92Binary files WebSearch/images/data/Google.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Google.png 2013-08-09 21:39:04 +0000 differ
93=== modified file 'WebSearch/images/data/ImageShack.png'
94Binary files WebSearch/images/data/ImageShack.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/ImageShack.png 2013-08-09 21:39:04 +0000 differ
95=== modified file 'WebSearch/images/data/Twitter.png'
96Binary files WebSearch/images/data/Twitter.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Twitter.png 2013-08-09 21:39:04 +0000 differ
97=== modified file 'WebSearch/images/data/Webshots.png'
98Binary files WebSearch/images/data/Webshots.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Webshots.png 2013-08-09 21:39:04 +0000 differ
99=== modified file 'WebSearch/images/data/Wikipedia.png'
100Binary files WebSearch/images/data/Wikipedia.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Wikipedia.png 2013-08-09 21:39:04 +0000 differ
101=== modified file 'WebSearch/images/data/Yahoo.png'
102Binary files WebSearch/images/data/Yahoo.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Yahoo.png 2013-08-09 21:39:04 +0000 differ
103=== modified file 'WebSearch/images/data/Youtube.png'
104Binary files WebSearch/images/data/Youtube.png 2010-05-29 23:12:40 +0000 and WebSearch/images/data/Youtube.png 2013-08-09 21:39:04 +0000 differ

Subscribers

People subscribed via source and target branches