Last modified: 2014-02-12 23:45:36 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T21935, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 19935 - Search box highlight
Search box highlight
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
Feature requests (Other open bugs)
unspecified
All All
: Lowest enhancement
: ---
Assigned To: Nobody - You can work on this!
: easy, patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-26 01:18 UTC by Michael
Modified: 2014-02-12 23:45 UTC (History)
14 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
text gets higlighted as soon as the focus is brought back to the text box (660 bytes, patch)
2012-02-11 11:39 UTC, akshay chugh
Details
Solved problem of the text be highlighted instead of the cursor going to the end of the word (4.65 KB, patch)
2012-02-11 11:46 UTC, Ashwini
Details

Description Michael 2009-07-26 01:18:39 UTC
Someone requested that when you click on the search box, the text be highlighted instead of the cursor going to the end of the word.
Comment 1 Derk-Jan Hartman 2009-10-19 14:14:10 UTC
We could add the following, but it would need to be tested on the iPhone.

$("#searchField").click(function() {
  $(this).focus()
  $(this).select()
})
Comment 2 Phil Chang 2011-10-06 00:03:12 UTC
Is this already implemented? If so, we can close this bug.
Comment 3 Patrick Reilly 2011-10-06 00:26:15 UTC
No it isn't implemented.
Comment 4 Sumana Harihareswara 2012-01-11 14:22:46 UTC
Quick question since you marked this as "easy" - could you point to the file(s) the newbie would need to fix?  Basically all the stylesheets in the extensions/MobileFrontend/stylesheets/ directory?
Comment 5 John Du Hart 2012-02-07 02:12:44 UTC
This needs to be implemented in application.js, however it can't (or shouldn't, really) use jQuery, but instead use native JavaScript.
Comment 6 akshay chugh 2012-02-11 11:39:40 UTC
Created attachment 9983 [details]
text gets higlighted as soon as the focus is brought back to the text box

It works fine with Mozilla Firefox and Internet Explorer 9, but it doesnt seem to work in google chrome.
Comment 7 Ashwini 2012-02-11 11:46:39 UTC
Created attachment 9984 [details]
Solved problem of the text be highlighted instead of the cursor going to the end of the word

solved the problem of the text be highlighted instead of the cursor going to the end of the word check out but its only work in IE and in Firefox. thanks to akhshay.
Comment 8 Sumana Harihareswara 2012-02-11 12:34:57 UTC
Hi, Ashwini and Akshay.  Thank you for the patches!  As you can see in 

https://www.mediawiki.org/wiki/How_to_become_a_MediaWiki_hacker#Submit_your_changes

in order to signal to the development community that this Bugzilla bug report has a patch associated with it that needs review, you should add the keywords

patch

and

need-review

I've now done this for you, but in the future, please do it yourself every time you add a patch in Bugzilla.  Thanks.

Have you run the automated test suites against your patch(es)?  That's a very good way to find problems.  https://www.mediawiki.org/wiki/Manual:PHP_unit_testing and https://www.mediawiki.org/wiki/Manual:PHP_unit_testing/Installing_PHPUnit will help you.  If you have any trouble, please find us in #mediawiki on Freenode and ask!
Comment 9 Arthur Richards 2012-02-11 12:43:28 UTC
(In reply to comment #8)
> Hi, Ashwini and Akshay.  Thank you for the patches!  As you can see in 
> 
> https://www.mediawiki.org/wiki/How_to_become_a_MediaWiki_hacker#Submit_your_changes
> 
> in order to signal to the development community that this Bugzilla bug report
> has a patch associated with it that needs review, you should add the keywords
> 
> patch
> 
> and
> 
> need-review

Unfortunately, they were not able to add the keywords. I assume this was a limitation of the user permissions for newly created accounts?
Comment 10 Sumana Harihareswara 2012-02-11 12:48:28 UTC
(In reply to comment #9)
> Unfortunately, they were not able to add the keywords. I assume this was a
> limitation of the user permissions for newly created accounts?

Yes, argh.  I've added the ability to edit keywords to Akshay's and Ashwini's Bugzilla accounts and will have to consider whether we should modify this lockdown to balance between between security and access/usability.
Comment 11 Jon 2012-02-22 13:10:14 UTC
akshay chugh and ashwini - thanks for your patches! I've applied them in r112103
Comment 12 Jon 2012-05-08 09:37:50 UTC
I've seen evidence of a user use her microphone to enter text into the search box. Then when she clicks on the search box to correct the mistake in the first word the entire text highlights. She then proceeds by clearing the entire text and typing from scratch. This evidence suggests we //shouldn't// highlight the search result.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links