| 36 | | ^(Describe your design here.)^ |
| | 34 | |
| | 35 | === Visibility === |
| | 36 | |
| | 37 | * Add Prop<Boolean> visible() in VisualElement; |
| | 38 | * Override the method in BaseVisualElement, like this: '''if (parent().get() == null) return true; return parent().get().visible().get() && computeVisible();'''. |
| | 39 | * This way, a visual element will not be visible unless all its parent are also visible. |
| | 40 | * Create '''protected boolean computeVisible()''' in BaseVisualElement, which returns true by default. This will cause some conflicts because of overrides, returning Boolean. Solve them :) |
| | 41 | * In most of the needed cases, this method is overriden already and no more work is needed. This considers halo menus. However, look around for possible problems. |
| | 42 | * In BoundControl, add '''protected abstract M getDefaultValue()'''. This will be implemented in subclasses, according to their type. It will return the value which will be used when nothing in computed. |
| | 43 | * Again in BoundControl, in computeModelData(), if the control is not visible, return the default value. |
| | 44 | * This way, a bound control will not recompute when not shown. |
| | 45 | |
| | 46 | === Halo menus === |
| | 47 | |
| | 48 | * Currently, the halo menus of a book contain about 170 bound controls. They are created and initialized every time a new BookDocView is opened. |
| | 49 | * If we want the halo menus to be in the main window, this means they will not have a parent of type PageWorkArea. Find all the '''findParentElement(PageWorkArea.class)''' and replace them with '''AppHaloUtil.getWorkArea()'''. The same about the static alternative of this method. |
| | 50 | * Find all '''findParentElement(BookDocView.class)''' and replace them with appropriate logic (for example, get the parent of the workArea). |
| | 51 | * Refactor the AppHalotil.getWorkArea(VisualElement element) so that it looks like: '''return AppViewUtil.getCurBookDocView(element).workArea().get()'''. |
| | 52 | * Move the prop haloMenus() from PageWorkArea to AppMainWindow. |
| | 53 | * Change the visualElementDefs of all the halo menus so that their parent will be of type AppMainWindow. |
| | 54 | * Move the part related to halo menus from the setup methods in the swingComponent() in PageWorkArea to a resource prop in AppMainWindow. |
| | 55 | * The new setup must firstly remove all the halo components from the halo layer of the last previous and the current PageWorkArea. Otherwise, the swing will not work properly. |
| | 56 | * Result: faster book opening, preview opening (about 5 sec), less memory usage. |
