Uploaded image for project: 'Jalview'
  1. Jalview
  2. JAL-3647

Jalview code should not call addNotify()

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 2.11.1_js
    • Fix Version/s: 2.11.1_js, 2.11.2
    • Component/s: awt-gui
    • Labels:
      None
    • Environment:
      Java+JavaScript

      Description

      Several classes in Jalview call addNotify(). This is strongly disrecommended, and in particular it is happening in AlignmentPanel before it can be laid out. This method is supposed to indicate that resources have been attached -- that the topLevelAncestor has an OS peer, allowing it access to fonts and graphics. If this is doing something important, it should be stated so in the code.

        Attachments

          Activity

            People

            Assignee:
            gmungoc Mungo Carstairs
            Reporter:
            hansonr Bob Hanson
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated: