Details

    • Type: Task
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Component/s: RCP
    • Labels:
      None

      Description

      The AddHost dialog must accomodate:

      a) unmapped Hosts.
      b) creation & edit cycle.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            gavin@codedragons.com Gavin Bong added a comment -

            My first attempt (see addHostDialog-firstAttempt.PNG) using org.eclipse.ui.dialogs.TwoPaneElementSelector is passable but does NOT allow multiple selections of HostGroups. So I'm junking this and starting afresh.

            I will use the Working set dialogs as inspiration.

            Show
            gavin@codedragons.com Gavin Bong added a comment - My first attempt (see addHostDialog-firstAttempt.PNG) using org.eclipse.ui.dialogs.TwoPaneElementSelector is passable but does NOT allow multiple selections of HostGroups. So I'm junking this and starting afresh. I will use the Working set dialogs as inspiration.
            Hide
            gavin@codedragons.com Gavin Bong added a comment -

            I cannibalised org.eclipse.ui.dialogs.TwoPaneElementSelector ( http://www.krugle.com/kse/codespaces/Cw69Pe ) and managed to make it work according to my needs.

            See an example in addHostDialog-working1.PNG

            In summary, TwoPaneElementSelector's second list widget can only allow single selects. We need multiple selections because a Host can belong to multiple HostGroups. Furthermore using checkboxes is more intuitive.
            Thus, I've replaced the SWT Table widget with CheckboxTableViewer ( http://www.krugle.com/kse/codespaces/DgWyDz ).

            Current issue: can we accomodate EPL 1.0 license ( http://www.eclipse.org/legal/epl-v10.html ) into radman's license ?

            Show
            gavin@codedragons.com Gavin Bong added a comment - I cannibalised org.eclipse.ui.dialogs.TwoPaneElementSelector ( http://www.krugle.com/kse/codespaces/Cw69Pe ) and managed to make it work according to my needs. See an example in addHostDialog-working1.PNG In summary, TwoPaneElementSelector's second list widget can only allow single selects. We need multiple selections because a Host can belong to multiple HostGroups. Furthermore using checkboxes is more intuitive. Thus, I've replaced the SWT Table widget with CheckboxTableViewer ( http://www.krugle.com/kse/codespaces/DgWyDz ). Current issue: can we accomodate EPL 1.0 license ( http://www.eclipse.org/legal/epl-v10.html ) into radman's license ?
            Hide
            gavin@codedragons.com Gavin Bong added a comment -

            Optimization on what nodes to refresh when a Host is created/updated.

            case #1
            ----------
            If an unmapped Host remains unmapped after an edit cycle,
            {
            refresh the "unmapped" tree node.
            }
            else
            refresh the whole Gateway.

            case #2
            -----------
            If an NON-unmapped Host remains NON-unmapped after an edit cycle,
            {
            refresh the "groups" tree node.
            }
            else
            refresh the whole Gateway.

            Show
            gavin@codedragons.com Gavin Bong added a comment - Optimization on what nodes to refresh when a Host is created/updated. case #1 ---------- If an unmapped Host remains unmapped after an edit cycle, { refresh the "unmapped" tree node. } else refresh the whole Gateway. case #2 ----------- If an NON-unmapped Host remains NON-unmapped after an edit cycle, { refresh the "groups" tree node. } else refresh the whole Gateway.

              People

              • Assignee:
                gavin@codedragons.com Gavin Bong
                Reporter:
                gavin@codedragons.com Gavin Bong
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Development