Changeset 4250

Timestamp:

01/10/14 11:25:24 (10 years ago)

Author:

twagoo

Message:

comments and readability of new CMDI view toggle solution

File:

• vlo/branches/to-wicket-1.6/vlo_web_app/src/main/java/eu/clarin/cmdi/vlo/pages/ShowResultPage.java (modified) (6 diffs)

vlo/branches/to-wicket-1.6/vlo_web_app/src/main/java/eu/clarin/cmdi/vlo/pages/ShowResultPage.java

@@ -503 +503 @@
     }
 
+    private Label completeCmdiLabel = null;
+    
     /**
      * Add complete CMDI view
@@ -509 +511 @@
      */
     private void addCompleteCmdiView(final SolrDocument solrDocument) {
-
+        // create a container for the complete CMDI view and a toggle link (this is required for proper AJAX updates)
         final MarkupContainer completeCmdiContainer = new WebMarkupContainer("completeCmdiContainer");
         completeCmdiContainer.setOutputMarkupId(true);
         add(completeCmdiContainer);
+        
+        // Add a toggle link that provides lazy execution of CMDI transformation
         Link toggleLink = new IndicatingAjaxFallbackLink("toggleCmdiView") {
 
@@ -518 +522 @@
             public void onClick(AjaxRequestTarget target) {
                 if (completeCmdiLabel == null) {
+                    // first click: perform transformation
                     createCompleteCmdiView(solrDocument);
                     completeCmdiContainer.addOrReplace(completeCmdiLabel);
                 } else {
+                    // subsequent click: toggle visibility of transformation output
                     completeCmdiLabel.setVisible(!completeCmdiLabel.isVisible());
                 }
@@ -526 +532 @@
             }
         };
+        // add a label to the toggle link that represents the visibility state of the transformation output
         final Label toggleLabel = new Label("toggleLabel", new AbstractReadOnlyModel<String>() {
 
@@ -539 +546 @@
         toggleLink.add(toggleLabel);
         completeCmdiContainer.add(toggleLink);
-        completeCmdiContainer.add(new WebMarkupContainer("completeCmdi"));
+        
+        // add a placeholder for the transformation
+        final WebMarkupContainer completeCmdiPlaceholder = new WebMarkupContainer("completeCmdi");
+        completeCmdiPlaceholder.setVisible(false);
+        completeCmdiContainer.add(completeCmdiPlaceholder);
     }
 
@@ -570 +581 @@
         completeCmdiLabel.setEscapeModelStrings(false);
     }
-
-    private Label completeCmdiLabel = null;
 }