swingx
  1. swingx
  2. SWINGX-187

JXHyperlink - active area too large

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.9.0
    • Fix Version/s: 0.9.3
    • Component/s: Misc Component
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      187

      Description

      Using swinglabs-demos from Oct 20, 2005, Hyperlinks demo

      Rolling over to the right of the SwingX hyperlink under "Links" causes SwingX
      hyperlink to underline, and will get activated on mouse click

      Hyperlink "active" area should be contained to the label area

        Activity

        Hide
        kleopatra added a comment -

        a related issue is the focus border: should it be painted around the text only?
        Currently it's painted around the button - which extends as far as the
        LayoutManager sizes it.

        taking Fred and Coleen in.

        Jeanette

        Show
        kleopatra added a comment - a related issue is the focus border: should it be painted around the text only? Currently it's painted around the button - which extends as far as the LayoutManager sizes it. taking Fred and Coleen in. Jeanette
        Hide
        rah003 added a comment -

        Remember what the decision was here? Do we want to paint around active text only
        now?

        Show
        rah003 added a comment - Remember what the decision was here? Do we want to paint around active text only now?
        Hide
        kleopatra added a comment -

        you mean the border? dont know, never got an answer. But think that would be the
        right-thing-to-do.

        Jeanette

        Show
        kleopatra added a comment - you mean the border? dont know, never got an answer. But think that would be the right-thing-to-do. Jeanette
        Hide
        kleopatra added a comment -

        started BasicHyperlinkUI:

        • replaced HandCursor with a subclass of BasicButtonUI (updates cursor on model
          changes)
        • overridden contains to return true only if over text/icon rect
        • changed paintFocus to draw focus border around text/icon rect

        for comments:
        http://forums.java.net/jive/thread.jspa?threadID=39462&tstart=0

        Show
        kleopatra added a comment - started BasicHyperlinkUI: replaced HandCursor with a subclass of BasicButtonUI (updates cursor on model changes) overridden contains to return true only if over text/icon rect changed paintFocus to draw focus border around text/icon rect for comments: http://forums.java.net/jive/thread.jspa?threadID=39462&tstart=0
        Hide
        kleopatra added a comment -

        build passed, nothing else I can think of (the hit-detection on the text/icon of
        a hyperlink still can't work because rollover support is per-cell), so closing

        Show
        kleopatra added a comment - build passed, nothing else I can think of (the hit-detection on the text/icon of a hyperlink still can't work because rollover support is per-cell), so closing
        Hide
        kleopatra added a comment -


        corrected target milestone to 0.9.x

        Show
        kleopatra added a comment - corrected target milestone to 0.9.x

          People

          • Assignee:
            kleopatra
            Reporter:
            anns97
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: