Merge lp:~jak-o/beeseek/focus into lp:beeseek/1.0

Proposed by Jacopo Moretto
Status: Merged
Approved by: Andrea Corbellini
Approved revision: 245
Merged at revision: not available
Proposed branch: lp:~jak-o/beeseek/focus
Merge into: lp:beeseek/1.0
Diff against target: None lines
To merge this branch: bzr merge lp:~jak-o/beeseek/focus
Reviewer Review Type Date Requested Status
Andrea Corbellini Approve
Review via email: mp+5075@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Andrea Corbellini (andrea.corbellini) wrote :

The JavaScript code added will be sent also with the configuration pages. I'll add a comment with this note. Apart this small issue, all is OK so I'm approving the merge proposal.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'webdata/html/search-form.html'
2--- webdata/html/search-form.html 2009-03-01 13:18:04 +0000
3+++ webdata/html/search-form.html 2009-03-31 16:59:40 +0000
4@@ -1,7 +1,7 @@
5 <div class="form">
6 <form action="/search" method="post">
7 <div>
8- <input class="textarea" type="text" name="keyword" value="$keywords" />
9+ <input id="focus" class="textarea" type="text" name="keyword" value="$keywords" />
10 <button class="find" type="submit">Find</button>
11 </div>
12 </form>
13
14=== modified file 'webdata/html/top.html'
15--- webdata/html/top.html 2009-02-28 19:50:48 +0000
16+++ webdata/html/top.html 2009-03-31 16:59:40 +0000
17@@ -8,8 +8,14 @@
18 <link rel="stylesheet" type="text/css" media="screen" href="/beeseek:style_basic.css" />
19 <meta name="description" content="Results from BeeSeek" />
20 <meta http-equiv="Content-type" content='text/html; charset="UTF-8"' />
21+ <script type="text/javascript">
22+ function getfocus()
23+ {
24+ document.getElementById('focus').focus();
25+ }
26+ </script>
27 </head>
28- <body>
29+ <body onload="getfocus()">
30
31 <div id="top">
32 <div id="links">

Subscribers

People subscribed via source and target branches