Commit 2e9e48bc authored by kmccullough@apple.com's avatar kmccullough@apple.com

2009-04-24 Kevin McCullough <kmccullough@apple.com>

        Reviewed by Dan Bernstein.

        We should not show the deletion UI for blockquotes in mail but they do 
        show now that we only check for one visible border.

        * editing/DeleteButtonController.cpp:
        (WebCore::isDeletableElement):



git-svn-id: svn://svn.chromium.org/blink/trunk@42852 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent cf45a5a7
2009-04-24 Kevin McCullough <kmccullough@apple.com>
Reviewed by Dan Bernstein.
We should not show the deletion UI for blockquotes in mail but they do
show now that we only check for one visible border.
* editing/DeleteButtonController.cpp:
(WebCore::isDeletableElement):
2009-04-24 Dan Bernstein <mitz@apple.com> 2009-04-24 Dan Bernstein <mitz@apple.com>
- address Hyatt's review comments on the last patch - address Hyatt's review comments on the last patch
...@@ -89,6 +89,9 @@ static bool isDeletableElement(const Node* node) ...@@ -89,6 +89,9 @@ static bool isDeletableElement(const Node* node)
if (node->hasTagName(ulTag) || node->hasTagName(olTag)) if (node->hasTagName(ulTag) || node->hasTagName(olTag))
return true; return true;
if (isMailBlockquote(node))
return false;
if (renderer->isPositioned()) if (renderer->isPositioned())
return true; return true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment