Ticket #2170 (closed tweak: obsolete)

Opened 10 years ago

Last modified 8 years ago

ui-refine-utilization

Reported by: deyan Owned by: kyli
Priority: critical Milestone: X3
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort: 2
Importance: 97 Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: deyan Design_owners: kyli
Imp._owners: kyli Test_owners:
Analysis_reviewers: kyli Changelog:
Design_reviewers: pap, meddle Imp._reviewers: meddle, deyan
Test_reviewers: Analysis_score: 4.5
Design_score: 4 Imp._score: 4
Test_score: 0

Description (last modified by deyan) (diff)

See attached screenshot.

1 Make the vertical flaps 160px wide (default size; each time Sophie starts, they should be this width)
2 Make the horizontal flap 160px tall (default size; each time Sophie starts, it should be this height)
3 Timeline flap

3.1 Put the object name in front of the timeline slider
3.2 Shrink the vertical gap between the top of the palette and the slider.
3.3 Make sure each time Sophie starts, the bottom flap is hidden.
3.4 Fix the "autohiding" zoom dropdown

4 Book window

4.1 Make the book window that large that the whole book is seen without scrollbars. (when opened)
4.2 Optional - Make the Book preview window as large as the edit window, if possible - on the same position
4.3 Optional - Currently, the book panel expands when the book window is large enough; when there is need of scrollbars, it shrinks. This is the opposite behaviour of what is needed. Fix this issue.

6 Optional - Define a visual language for buttons - we need to separate the toggle buttons (filters, toggle frames) from push buttons (Import, Open preview).

6.1 - Optional - Change graphics of affected buttons.

7 Optional - Make the "Pin/Unpin" tab buttons to affect all the tabs on this side (right, left and bottom)

Attachments

ui_optimized.jpg (133.2 KB) - added by deyan 10 years ago.

Change History

Changed 10 years ago by deyan

comment:1 Changed 10 years ago by deyan

  • Category set to unknown
  • Design_score set to 0
  • Imp._score set to 0
  • Test_score set to 0
  • Analysis_score set to 0

See attached screenshot.

1 Make the vertical flaps 160px wide (default size; each time Sophie starts, they should be this width)
2 Make the horizontal flap 160px tall (default size; each time Sophie starts, it should be this height)
3 Flaps
4 Timeline flap

4.1 Put the object name in front of the timeline slider
4.2 Shrink the vertical gap between the top of the palette and the slider.
4.3 Make sure each time Sophie starts, the bottom flap is hidden.

comment:2 Changed 10 years ago by deyan

  • Owner set to deyan
  • Status changed from new to s1a_analysis_started

comment:3 Changed 10 years ago by deyan

  • Description modified (diff)

comment:4 Changed 10 years ago by deyan

  • Description modified (diff)

comment:5 Changed 10 years ago by deyan

  • Priority changed from major to critical

Batch update from file 0911261.csv

comment:6 Changed 10 years ago by kyli

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:7 Changed 10 years ago by kyli

  • Design_owners set to kyli
  • Status changed from s1b_analysis_finished to s1c_analysis_ok

comment:8 Changed 10 years ago by kyli

  • Owner changed from deyan to kyli
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:9 Changed 10 years ago by kyli

  • Status changed from s2a_design_started to new

Sorry, I think I will not be able to finish this task in time, so I drop it. However, the analysis seems unfinished - the 7th point of the requirements is not consistent.

comment:10 Changed 10 years ago by deyan

  • Owner changed from kyli to deyan
  • Status changed from new to s1a_analysis_started
  • Description modified (diff)

comment:11 Changed 10 years ago by deyan

  • Status changed from s1a_analysis_started to s1b_analysis_finished
  • Analysis_owners set to deyan

comment:12 Changed 10 years ago by deyan

  • Description modified (diff)

comment:13 Changed 10 years ago by kyli

  • Status changed from s1b_analysis_finished to s1c_analysis_ok

comment:14 Changed 10 years ago by kyli

  • Owner changed from deyan to kyli
  • Status changed from s1c_analysis_ok to s2a_design_started
  • Analysis_reviewers set to kyli

comment:15 Changed 10 years ago by kyli

  • Status changed from s2a_design_started to s2b_design_finished

In order to utilize space, flaps will be removed.

  • The class !MDFlap is useful itself, but it should only take care of its tabs. It is good for automated creation of !MDTabs as well as determining their anchor. So, it will neither extend MDCompoundLayoutElement anymore, nor have a swingComponent.
  • The !MDMainWindow will create !MDFlaps, but its registerFlap(..) method will actually register the flap's tabs. The repositionFlap(..) method should be much more simple, since it is too complicated and even doesn't work good enough. It will work with tabs instead of flaps, too. Many of the @Setup methods in this class must be revised - the inner prop reads should be minimized.
  • MDCompoundLayoutElement will be removed. Its code will be moved to MDTab.
  • MDTab will not have a swingComponent - its toolWindowManager is enough. Instead, the toolWindowManager will be changed to resource prop, so that it will have the swingComponent's code in its setup method. In the main window, only the toolWindowManager will be registered - the way it is in the MyDoggy tutorial. This resolves some problems..
  • MDToolWindowFactory is currently useless. Change the value of setDockLength and see it yourself :) It seems that the tool window configuration should be as in the MyDoggy tutorial, otherwise it has no effect or the effect is not the desired one. Unfortunately, this means that tab animation should be enabled - otherwise, tabs are shown with random initial size. This results in some kind of a bad experience with the left flap..
  • Look around the MD module and fix/remove unused code, which appears to be a lot (for example, add/remove layout element, complicated hierarchies, etc).
  • The initial book window's size is hold in MDMainWindow.INTERNAL_FRAME_SIZE field. Change it to 700x650 in order to display a book with default size of 640x480.
  • In the TimelinesPalette, play with the cornerPanel's layout in order to center the label in front of the slider. BoxLayout.CENTER is good enough for this. The slider's size can be shrinked with 15px (if more, the slider starts to disappear). This will be done.
  • The bug with the zoom combo is resolved as soon as flaps are removed - it seems, that the wrong usage of myDoggy for the bottom flap causes the elements to steal focus each other's.
  • The size of the book preview window is already fixed in the trunk.
  • For the other optional requirements, there was not enough time.. sorry for that :)

comment:16 Changed 10 years ago by meddle

  • Status changed from s2b_design_finished to s2c_design_ok
  • Design_score changed from 0 to 4
  • Design_reviewers set to pap, meddle
  • Analysis_score changed from 0 to 4.5
  • It's good that you erased a lot of unusable code.
  • The bug with the zoom combo is solved, that is awesome.
  • There is no design related code... to look at it :(
  • You should have explained why 700x650 makes the book's default size 640x480
  • Boooo :)
  • You should have marked the analysis of Deyan... I think it's vey good -> 4.5p

4p

comment:17 Changed 10 years ago by kyli

  • Status changed from s2c_design_ok to s3a_implementation_started
  • Imp._owners set to kyli
  • Boo boo!

comment:18 Changed 10 years ago by kyli

  • Status changed from s3a_implementation_started to s3b_implementation_finished

The implementation is done in http://sophie2.org/trac/browser/branches/private/kyli/UI, according to the design. Some notes:

  • The internal frame size was determined experimentally.
  • Erasing all this code could probably mean that I've deleted some bugfixes, so please test my code very carefully :) Thanks. It currently is merged with trunk, builds with maven, True/Fake author and reader, as well as fake applet configurations run.
  • Additional things done:
    • The tabs now have default top padding ot 25px (experimentally determined).
    • Timeline zoom combo is working.
    • The timeline slider's tick spacing was fixed.
    • Fixed the scenario in which you select the book desktop and all the hidden flaps appear.
    • A book extras related palette which was not used is deleted.
    • Selecting book desktop now does not re-setup the global container (previously, it made it twice each time).
    • The book extras tab's name was changed from "BookExtras" to "Book extras".
    • A WRAP_MODE warning in FrameH was removed, since in preview mode it prints every 20ms. It is annoying :)
    • Fixed the bottom flap overlapping the book window when toggling full-screen mode. Actually, it currently remembers the sizes of all the tabs and restores them when going to normal mode (a smart ass).
  • Tickets #2012 and #2014 (which are related to this) are currently obsolete, since palettes do not have pin/float buttons.

comment:19 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 0 to 4
  • Imp._reviewers set to meddle, deyan

Merged to the trunk at [8518], just remember that var_x, is not a name for variable when writing with our conventions.

4p

comment:20 Changed 8 years ago by meddle

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to obsolete

Closing all the tickets before M Y1

Note: See TracTickets for help on using tickets.