You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@isis.apache.org by da...@apache.org on 2014/12/02 08:21:04 UTC

[2/2] isis git commit: ISIS-819: move "development" warning to be part of the after the about link, rather than the powered by link.

ISIS-819: move "development" warning to be part of the after the about link, rather than the powered by link.


Project: http://git-wip-us.apache.org/repos/asf/isis/repo
Commit: http://git-wip-us.apache.org/repos/asf/isis/commit/e78b950a
Tree: http://git-wip-us.apache.org/repos/asf/isis/tree/e78b950a
Diff: http://git-wip-us.apache.org/repos/asf/isis/diff/e78b950a

Branch: refs/heads/master
Commit: e78b950a086c8501c9d792e064377b122ccfc24f
Parents: 0092b32
Author: Dan Haywood <da...@haywood-associates.co.uk>
Authored: Tue Dec 2 07:19:26 2014 +0000
Committer: Dan Haywood <da...@haywood-associates.co.uk>
Committed: Tue Dec 2 07:19:26 2014 +0000

----------------------------------------------------------------------
 .../isis/viewer/wicket/ui/pages/PageAbstract.html    | 10 ++++++++--
 .../isis/viewer/wicket/ui/pages/PageAbstract.java    | 15 +++++++++------
 2 files changed, 17 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/isis/blob/e78b950a/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.html
----------------------------------------------------------------------
diff --git a/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.html b/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.html
index 54396dc..5cd3c10 100644
--- a/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.html
+++ b/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.html
@@ -96,8 +96,14 @@
                             </ul>
                             <ul class="nav navbar-nav navbar-right">
                                 <p class="navbar-text powered-by">Powered by:</p>
-                                <li><a href="http://isis.apache.org" target="_blank">Apache Isis &#8482; <i wicket:id="devModeWarning" class="fa fa-lg fa-warning text-danger" title="Running in Development mode!"></i></a></li>
-                                <li><a wicket:id="aboutLink" class="navbar-right">About</a></li>
+                                <li><a href="http://isis.apache.org" target="_blank">Apache Isis &#8482;</a></li>
+                                <li>
+                                    <a wicket:id="aboutLink" class="navbar-right">
+                                        <span wicket:id="aboutMessage">About</span>
+                                        <i wicket:id="devModeWarning" class="fa fa-lg fa-warning text-danger" title="Running in Prototype (Development) mode!"/>
+                                    </a>
+
+                                </li>
                                 <li wicket:id="themePicker"></li>
                             </ul>
                         </div>

http://git-wip-us.apache.org/repos/asf/isis/blob/e78b950a/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.java
----------------------------------------------------------------------
diff --git a/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.java b/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.java
index 43886d2..0d7b129 100644
--- a/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.java
+++ b/component/viewer/wicket/ui/src/main/java/org/apache/isis/viewer/wicket/ui/pages/PageAbstract.java
@@ -181,7 +181,7 @@ public abstract class PageAbstract extends WebPage implements ActionPromptProvid
             addUserName(themeDiv);
             addLogoutLink(themeDiv);
             addAboutLink(themeDiv);
-            addDevModeWarning(themeDiv);
+
             addBreadcrumbs();
             addThemePicker();
 
@@ -296,18 +296,21 @@ public abstract class PageAbstract extends WebPage implements ActionPromptProvid
 
     private void addAboutLink(MarkupContainer themeDiv) {
         BookmarkablePageLink<Void> aboutLink = new BookmarkablePageLink<>("aboutLink", AboutPage.class);
-        aboutLink.setBody(new ResourceModel("aboutLabel"));
         themeDiv.add(aboutLink);
+
+        Label aboutLabel = new Label("aboutMessage", new ResourceModel("aboutLabel"));
+        aboutLink.add(aboutLabel);
+        addDevModeWarning(aboutLink);
     }
 
     /**
-     * Adds a component that shows a warning sign next to "Powered by: Apache Isis" in development mode
-     * @param themeDiv The parent component
+     * Adds a component that shows a warning sign next to "About" link in development mode
+     * @param container The parent component
      */
-    private void addDevModeWarning(MarkupContainer themeDiv) {
+    private void addDevModeWarning(MarkupContainer container) {
         WebComponent devModeWarning = new WebComponent("devModeWarning");
         devModeWarning.setVisible(getApplication().usesDevelopmentConfig());
-        themeDiv.add(devModeWarning);
+        container.add(devModeWarning);
     }
 
     private void addBreadcrumbs() {