cayenne-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nikita Timofeev (Jira)" <j...@apache.org>
Subject [jira] [Updated] (CAY-2680) Text Fields in Modeler loose modification when switching to another tab or panel
Date Wed, 24 Feb 2021 10:00:00 GMT

     [ https://issues.apache.org/jira/browse/CAY-2680?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Nikita Timofeev updated CAY-2680:
---------------------------------
    Fix Version/s:     (was: 4.2.M3)
                   4.3.M1

> Text Fields in Modeler loose modification when switching to another tab or panel
> --------------------------------------------------------------------------------
>
>                 Key: CAY-2680
>                 URL: https://issues.apache.org/jira/browse/CAY-2680
>             Project: Cayenne
>          Issue Type: Bug
>          Components: Modeler
>    Affects Versions: 4.1
>            Reporter: Stefan S.
>            Assignee: Nikita Timofeev
>            Priority: Minor
>             Fix For: 4.1.1, 4.3.M1
>
>         Attachments: cayenne_modeler_text_adapter_patch.diff
>
>
> Observation:
> Several (single line) text fields in Modeler forget changes or insertions of text if
the user switches to a different panel or tab without explicitly leaving (bluring) the text
field.
>  
> Expectation:
> Text changes shall be kept also in those cases.
>  
> Background:
> It hit me several times, when, e.g. walking through a list of entities to add comments
or modify other texts, that the changes are dismissed due to switching to the next entity
or a different 'area' of the application. To keep the changes, it is necessary to click onto
a different UI element within the same panel. It seems that changes of single line texts are
'taken over' only on 'focus lost' event, which are not fired in the described cases.
>  
> As a simple solution, I defaulted 'checkOnTyping' to 'true' in the standard constructor
of
> {code:java}
> org.apache.cayenne.modeler.util.TextAdapter
> {code}
> which is more a quick hack, since it might not cover all relevant instances and introduces
some potential processing overhead when typing. A better approach would probably be to trigger
the commit of changed texts in listeners of appropriate events. Nevertheless no negative impact
to performance was observed using this approach.
> The attached patch provides this change AND the propagation of constructor arguments,
which have originally been dismissed and replaced by hardcode values.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message