[Qt] Disable focus ring in the mobile theme.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Mar 2012 14:49:18 +0000 (14:49 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Mar 2012 14:49:18 +0000 (14:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=81934

For mobile devices intended to be used only through
touch, the focus rings makes little sense and feels
awkward.

Patch by Pierre Rossi <pierre.rossi@nokia.com> on 2012-03-26
Reviewed by Kenneth Rohde Christiansen.

No new tests. This is merely a cosmetic touch.

* platform/qt/RenderThemeQtMobile.h:
(RenderThemeQtMobile):
(WebCore::RenderThemeQtMobile::platformFocusRingColor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@112095 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/qt/RenderThemeQtMobile.h

index 9a19b733789b6804596e0f81a3a9b652af74193c..58bb8cf6dd201734eb3913d063539e155d8be436 100644 (file)
@@ -1,3 +1,20 @@
+2012-03-26  Pierre Rossi  <pierre.rossi@nokia.com>
+
+        [Qt] Disable focus ring in the mobile theme.
+        https://bugs.webkit.org/show_bug.cgi?id=81934
+
+        For mobile devices intended to be used only through
+        touch, the focus rings makes little sense and feels
+        awkward.
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        No new tests. This is merely a cosmetic touch.
+
+        * platform/qt/RenderThemeQtMobile.h:
+        (RenderThemeQtMobile):
+        (WebCore::RenderThemeQtMobile::platformFocusRingColor):
+
 2012-03-26  Charles Wei  <charles.wei@torchmobile.com.cn>
 
         [BlackBerry] Clean up networking code and remove WML support.
 2012-03-26  Charles Wei  <charles.wei@torchmobile.com.cn>
 
         [BlackBerry] Clean up networking code and remove WML support.
index 0d36b8c931e057bf196a17938d15274fa13761f2..1c2480f35ea154afc1040deb7c0cbcb474a823f5 100644 (file)
@@ -53,6 +53,9 @@ public:
 
     virtual bool delegatesMenuListRendering() const { return true; }
 
 
     virtual bool delegatesMenuListRendering() const { return true; }
 
+    // drawFocusRing() will return early if the color is invalid.
+    virtual Color platformFocusRingColor() const { return Color(); }
+
 protected:
 
     virtual void adjustButtonStyle(CSSStyleSelector*, RenderStyle*, Element*) const;
 protected:
 
     virtual void adjustButtonStyle(CSSStyleSelector*, RenderStyle*, Element*) const;