[webkit-reviews] review granted: [Bug 208157] Prevent text selection on logo and header navigation : [Attachment 391580] Patch

[bugzilla-daemon at webkit.org]

Devin Rousso <drousso at apple.com> has granted Jon Davis <jond at apple.com>'s
request for review:
Bug 208157: Prevent text selection on logo and header navigation
https://bugs.webkit.org/show_bug.cgi?id=208157

Attachment 391580: Patch

https://bugs.webkit.org/attachment.cgi?id=391580&action=review




--- Comment #2 from Devin Rousso <drousso at apple.com> ---
Comment on attachment 391580
  --> https://bugs.webkit.org/attachment.cgi?id=391580
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=391580&action=review

r=me

> Websites/webkit.org/wp-content/themes/webkit/style.css:2103
>  header nav {

Why not put this on `header`?

Right now (with this patch), the "WebKit" next to the logo is still selectable.
 Do we want that?