SciNotes: make a little cleanup 45/14045/2
Calixte DENIZET [Mon, 17 Mar 2014 12:41:28 +0000 (13:41 +0100)]
Change-Id: Id38fed6c0659309532bd72d8671c8411183c0fa1

scilab/modules/scinotes/etc/scinotesConfiguration.xml
scilab/modules/scinotes/src/java/org/scilab/modules/scinotes/SciNotes.java
scilab/modules/scinotes/src/java/org/scilab/modules/scinotes/actions/SuppressCommentsAction.java [deleted file]
scilab/modules/scinotes/src/java/org/scilab/modules/scinotes/utils/ConfigSciNotesManager.java

index 9e2fe4e..4ab8126 100644 (file)
  *
  -->
 
-<Setting version="0.41">
+<Setting version="0.42">
     
     <!-- SCINOTES configuration -->
     <Profile name="scinotes">
         
-        <!-- GUI Part -->
-        
-        <ScrollableUnitIncrement value="20"/>
-        <ScrollableBlockIncrement value="20"/>
-        <HorizontalWrapAllowed default="true" value="true"/>
-        <AutoIndent default="true" value="true"/>
-        <AutoColorize default="true" value="true"/>
-        <DefaultEncoding value="utf-8"/>
-        <!-- Initialized to system default at first launch -->
-        
-        
-        <!-- linecolor or contourcolor can be "null" to be transparent -->
-        <LineHighlighter value="false" linecolor="#e4e9f4" contourcolor="#323232"/>
-        
         <!-- colors to use in the left panel to indicate you're a in a function (color1)
         or in a inner function (color2) (only available in whereami mode)-->
         <AlternColors color1="#f6bff6" color2="#f665f6"/>
         
-        <!-- Activate or not the help on typing for openers such as '(', '[', '"',... -->
-        <HelpOnTypingOpeners value="true"/>
-        
-        <!-- Activate or not the help on typing for keywords such as 'if', 'function',... -->
-        <HelpOnTypingKeywords value="true"/>
-        
         <!-- Restore the open files from the last session  -->
         <RestoreFiles value="true"/>
         
-        <!-- Line numbering : 0 is off, 1 is normal and 2 is whereami-->
-        <LineNumbering value="1"/>
-        
-        <!-- Suppress comments when executing code with ctrl+E -->
-        <SuppressComments value="true"/>
-        
-        <!-- * inside is a boolean to indicate if the contents between the keywords must be highlighted.
-         * included is only used when inside is true to say if keywords or open/close must be highlighted.
-         * If inside is true type can be filled or framed else it can be filled, framed or underlined.
-         * strict is only used for Keywords when inside is true, in this case the lines which contain the keywords
-           are not highlighted. -->
-        <Matching onmouseover="true">
-            <KeywordsHighlighter color="#fff3d2" inside="true" included="false" strict="true" type="filled"/>
-            <OpenCloseHighlighter color="#e40087" inside="false" included="true" type="framed"/>
-        </Matching>
-        
-        <SciNotes>
-            <BackgroundColor default="#FFFFFF" value="#FFFFFF"/>
-            <ForegroundColor default="#000000" value="#000000"/>
-        </SciNotes>
-        
-        <styles>
-            
-            <FontName default="Monospaced" value="Monospaced"/>
-            <FontSize default="14" value="14"/>
-            
-            <MainWindowPosition x="150" y="50"/>
-            <MainWindowSize height="550" width="650"/>
-            
-            <style name="Default">
-                <ForegroundColor default="#000000" value="#000000"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Special">
-                <ForegroundColor default="#ffaa00" value="#ffaa00"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Url" underline="true" defaultUnderline="true">
-                <ForegroundColor default="#0000ff" value="#0000ff"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Mail" underline="true" defaultUnderline="true">
-                <ForegroundColor default="#0000ff" value="#0000ff"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Constantes">
-                <ForegroundColor default="#da70d6" value="#da70d6"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Operator">
-                <ForegroundColor default="#5c5c5c" value="#5c5c5c"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Number">
-                <ForegroundColor default="#bc8f8f" value="#bc8f8f"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="FunctionKwd">
-                <ForegroundColor default="#b01813" value="#b01813"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Structure">
-                <ForegroundColor default="#a020f0" value="#a020f0"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Control">
-                <ForegroundColor default="#5f9ea0" value="#5f9ea0"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Primitive">
-                <ForegroundColor default="#32b9b9" value="#32b9b9"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Macro" underline="true" defaultUnderline="true">
-                <ForegroundColor default="#ae5cb0" value="#ae5cb0"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="MacroInFile" underline="true" defaultUnderline="true">
-                <ForegroundColor default="#000000" value="#000000"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="String">
-                <ForegroundColor default="#bc8f8f" value="#bc8f8f"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="ExternalVariable">
-                <ForegroundColor default="#FF0000" value="#FF0000"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="InputOutputArgs">
-                <ForegroundColor default="#834310" value="#834310"/>
-                <FontStyle default="1" value="1"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Identifier">
-                <ForegroundColor default="#000000" value="#000000"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Comment">
-                <ForegroundColor default="#01a801" value="#01a801"/>
-                <FontStyle default="2" value="2"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="WhiteInComment">
-                <ForegroundColor default="#90f790" value="#90f790"/>
-                <FontStyle default="2" value="2"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="TabulationInComment">
-                <ForegroundColor default="#90f790" value="#90f790"/>
-                <FontStyle default="2" value="2"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="LaTeXInComment">
-                <ForegroundColor default="#8b2252" value="#8b2252"/>
-                <FontStyle default="2" value="2"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="WhiteInString">
-                <ForegroundColor default="#e9b1b1" value="#e9b1b1"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="TabulationInString">
-                <ForegroundColor default="#e9b1b1" value="#e9b1b1"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="Field">
-                <ForegroundColor default="#aaaaaa" value="#aaaaaa"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <!-- rep can be "doublechevrons", "horizontal" or "vertical" or a single char such as "#" or "none" -->
-            <!-- value is the number of space equivalent to a tabulation -->
-            <!-- white is a boolean indicating if a \t must be inserted (false) or a sequence of whites -->
-            <style name="Tabulation" rep="doublechevrons" value="4" white="true">
-                <ForegroundColor default="#dcdcdc" value="#dcdcdc"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="White">
-                <ForegroundColor default="#dcdcdc" value="#dcdcdc"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-            
-            <style name="OpenClose">
-                <ForegroundColor default="#4a55db" value="#4a55db"/>
-                <FontStyle default="0" value="0"/>
-                <!-- 0=PLAIN, 1=BOLD, 2=ITALIC, 3=BOLD+ITALIC -->
-            </style>
-        </styles>
-        
         <!-- Paper format Section
         The default unit is the postscript point (1/72 inch) -->
         <PaperFormat MarginBottom="58.0" MarginLeft="57.0" MarginRight="57.0" MarginTop="57.0" height="794.0" width="613.0"/>
         <!-- Currently Open Files Section  -->
         <openFiles>
         </openFiles>
-        
     </Profile>
 </Setting>
index 8963ee6..b311e5d 100644 (file)
@@ -692,6 +692,7 @@ public class SciNotes extends SwingScilabDockablePanel {
             if (getTabPane().getTabCount() != 1 || getTextPane(0).getName() != null) {
                 openFile(null, 0, null);
             }
+
             setWindowIcon("accessories-text-editor");
             WindowsConfigurationManager.restorationFinished(this);
 
@@ -1939,25 +1940,6 @@ public class SciNotes extends SwingScilabDockablePanel {
     }
 
     /**
-     * Auto-indent mode management
-     *
-     * @param b
-     *            true to activate auto-indent mode
-     */
-    public static void setSuppressComments(boolean b) {
-        for (SciNotes ed : scinotesList) {
-            int n = ed.getTabPane().getTabCount();
-            for (int i = 0; i < n; i++) {
-                ScilabEditorPane sep = ed.getTextPane(i);
-                sep.suppressCommentsInExecutingCode(b);
-                if (sep.getOtherPaneInSplit() != null) {
-                    sep.getOtherPaneInSplit().suppressCommentsInExecutingCode(b);
-                }
-            }
-        }
-    }
-
-    /**
      * Horizontal Wrap mode management
      *
      * @param b
diff --git a/scilab/modules/scinotes/src/java/org/scilab/modules/scinotes/actions/SuppressCommentsAction.java b/scilab/modules/scinotes/src/java/org/scilab/modules/scinotes/actions/SuppressCommentsAction.java
deleted file mode 100644 (file)
index 833a047..0000000
+++ /dev/null
@@ -1,77 +0,0 @@
-/*
- * Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
- * Copyright (C) 2011 - Calixte DENIZET
- *
- * This file must be used under the terms of the CeCILL.
- * This source file is licensed as described in the file COPYING, which
- * you should have received as part of this distribution.  The terms
- * are also available at
- * http://www.cecill.info/licences/Licence_CeCILL_V2.1-en.txt
- *
- */
-
-package org.scilab.modules.scinotes.actions;
-
-import java.beans.PropertyChangeEvent;
-import java.beans.PropertyChangeListener;
-
-import javax.swing.JCheckBoxMenuItem;
-import javax.swing.KeyStroke;
-import javax.swing.SwingUtilities;
-
-import org.scilab.modules.gui.checkboxmenuitem.CheckBoxMenuItem;
-import org.scilab.modules.scinotes.SciNotes;
-import org.scilab.modules.scinotes.utils.ConfigSciNotesManager;
-
-/**
- * SuppressCommentsAction Class
- * @author Calixte DENIZET
- *
- */
-public final class SuppressCommentsAction extends DefaultCheckAction  {
-
-    /**
-     * serialVersionUID
-     */
-    private static final long serialVersionUID = -1L;
-
-    /**
-     * Constructor
-     * @param name the name of the action
-     * @param editor SciNotes
-     */
-    public SuppressCommentsAction(String name, SciNotes editor) {
-        super(name, editor);
-    }
-
-    /**
-     * doAction
-     */
-    public void doAction() {
-        SwingUtilities.invokeLater(new Runnable() {
-            public void run() {
-                SciNotes.setSuppressComments(SuppressCommentsAction.this.getState());
-                ConfigSciNotesManager.saveSuppressComments(SuppressCommentsAction.this.getState());
-            }
-        });
-    }
-
-    /**
-     * createCheckBoxMenu
-     * @param label label of the menu
-     * @param editor SciNotes
-     * @param key KeyStroke
-     * @return CheckBoxMenuItem
-     */
-    public static CheckBoxMenuItem createMenu(String label, SciNotes editor, KeyStroke key) {
-        final CheckBoxMenuItem sca = createCheckBoxMenu(label, null, new SuppressCommentsAction(label, editor), key);
-        sca.setChecked(ConfigSciNotesManager.getSuppressComments());
-        ((JCheckBoxMenuItem) sca.getAsSimpleCheckBoxMenuItem()).addPropertyChangeListener(new PropertyChangeListener() {
-            public void propertyChange(PropertyChangeEvent e) {
-                sca.setChecked(ConfigSciNotesManager.getSuppressComments());
-            }
-        });
-
-        return sca;
-    }
-}
index ca35b31..4aa293d 100644 (file)
@@ -84,7 +84,6 @@ public final class ConfigSciNotesManager {
 
     private static final int MARGIN = 20;
 
-    private static final String HORIZONTALWRAP = "HorizontalWrapAllowed";
     private static final String ERROR_READ = "Could not load file: ";
     private static final String ERROR_WRITE = "Could not save file: ";
     private static final String VALUE = "value";
@@ -110,8 +109,7 @@ public final class ConfigSciNotesManager {
     private static final String HELPONTYPING = "HelpOnTyping";
     private static final String LINENUMBERING = "LineNumbering";
     private static final String EDITOR = "SciNotes";
-    private static final String SUPPRESSCOMMENTS = "SuppressComments";
-
+    
     private static final String FOREGROUNDCOLOR = "ForegroundColor";
     private static final String BACKGROUNDCOLOR = "BackgroundColor";
     private static final String ALTERNCOLORS = "AlternColors";
@@ -360,138 +358,6 @@ public final class ConfigSciNotesManager {
     }
 
     /**
-     * Get the background Color
-     * @return the background Color
-     */
-    public static Color getSciNotesBackgroundColor() {
-        /* Load file */
-        readDocument();
-
-        Element root = document.getDocumentElement();
-
-        NodeList profiles = root.getElementsByTagName(EDITOR);
-        Element scinotesProfile = (Element) profiles.item(0);
-
-        NodeList allElements = scinotesProfile.getElementsByTagName(BACKGROUNDCOLOR);
-        Element scinotesBackground = (Element) allElements.item(0);
-
-        /*direct create a Color with "#FF00FF" string from the xml */
-        return Color.decode(scinotesBackground.getAttribute(VALUE));
-    }
-
-    /**
-     * Get the foreground Color
-     * @return the foreground Color
-     */
-    public static Color getSciNotesForegroundColor() {
-        /* Load file */
-        readDocument();
-
-        Element root = document.getDocumentElement();
-
-        NodeList profiles = root.getElementsByTagName(EDITOR);
-        Element scinotesProfile = (Element) profiles.item(0);
-
-        NodeList allElements = scinotesProfile.getElementsByTagName(FOREGROUNDCOLOR);
-        Element scinotesForeground = (Element) allElements.item(0);
-
-        /*direct create a Color with "#FF00FF" string from the xml */
-        return Color.decode(scinotesForeground.getAttribute(VALUE));
-    }
-
-    /**
-     * Save SciNotes BackgroundColor
-     * @param color the new Color
-     */
-    public static void saveSciNotesBackground(Color color) {
-
-        /* Load file */
-        readDocument();
-
-        Element root = document.getDocumentElement();
-
-        NodeList profiles = root.getElementsByTagName(EDITOR);
-        Element scinotesProfile = (Element) profiles.item(0);
-
-        NodeList allSizeElements = scinotesProfile.getElementsByTagName(BACKGROUNDCOLOR);
-        Element scinotesBackground = (Element) allSizeElements.item(0);
-
-        String rgb = Integer.toHexString(color.getRGB());
-        scinotesBackground.setAttribute(VALUE, COLORPREFIX + rgb.substring(2, rgb.length()));
-
-        /* Save changes */
-        writeDocument();
-    }
-
-    /**
-     * Save SciNotes foregroundColor
-     * @param color the new Color
-     */
-    public static void saveSciNotesForeground(Color color) {
-        readDocument();
-
-        Element root = document.getDocumentElement();
-
-        NodeList profiles = root.getElementsByTagName(EDITOR);
-        Element scinotesProfile = (Element) profiles.item(0);
-
-        NodeList allSizeElements = scinotesProfile.getElementsByTagName(FOREGROUNDCOLOR);
-        Element scinotesForeground = (Element) allSizeElements.item(0);
-
-        String rgb = Integer.toHexString(color.getRGB());
-        scinotesForeground.setAttribute(VALUE, COLORPREFIX + rgb.substring(2, rgb.length()));
-
-        /* Save changes */
-        writeDocument();
-    }
-
-    /**
-     * Save SciNotes autoIndent or not
-     * @param activated if autoIndent should be used or not
-     */
-    public static void saveSuppressComments(boolean activated) {
-        /* Load file */
-        readDocument();
-
-        Element root = document.getDocumentElement();
-
-        NodeList profiles = root.getElementsByTagName(PROFILE);
-        Element scinotesProfile = (Element) profiles.item(0);
-
-        NodeList allSizeElements = scinotesProfile.getElementsByTagName(SUPPRESSCOMMENTS);
-        Element suppressComments = (Element) allSizeElements.item(0);
-        if (suppressComments == null) {
-            Element sup = document.createElement(SUPPRESSCOMMENTS);
-            sup.setAttribute(VALUE, new Boolean(activated).toString());
-            scinotesProfile.appendChild((Node) sup);
-        } else {
-            suppressComments.setAttribute(VALUE, new Boolean(activated).toString());
-        }
-        /* Save changes */
-        writeDocument();
-    }
-
-    /**
-     * @return a boolean if autoIndent should be used or not
-     */
-    public static boolean getSuppressComments() {
-        /* Load file */
-        readDocument();
-
-        Element root = document.getDocumentElement();
-        NodeList profiles = root.getElementsByTagName(PROFILE);
-        Element scinotesProfile = (Element) profiles.item(0);
-        NodeList allSizeElements = scinotesProfile.getElementsByTagName(SUPPRESSCOMMENTS);
-        Element suppressComments = (Element) allSizeElements.item(0);
-
-        if (suppressComments == null) {
-            return true;
-        } else {
-            return new Boolean(suppressComments.getAttribute(VALUE));
-        }
-    }
-
-    /**
      * Get all the recent opened files
      * @return an array of uri
      */