Ticket #1966 (closed bug: fixed)
Book desktop should be in the opened books palette
Reported by: | deyan | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | M12_RELEASE |
Component: | uncategorized | Version: | 2.0 |
Keywords: | Cc: | ||
Category: | unknown | Effort: | |
Importance: | Ticket_group: | ||
Estimated Number of Hours: | 0 | Add Hours to Ticket: | 0 |
Billable?: | yes | Total Hours: | 0 |
Analysis_owners: | deyan | Design_owners: | mitex |
Imp._owners: | mitex | Test_owners: | deyan |
Analysis_reviewers: | mitex | Changelog: | |
Design_reviewers: | Imp._reviewers: | mira | |
Test_reviewers: | Analysis_score: | 3 | |
Design_score: | 3.5 | Imp._score: | 3.5 |
Test_score: | 4 |
Description
Book desktop is not currently available in opened books palette, it may be shown there as "Show Desktop" for example
Change History
comment:2 Changed 16 years ago by mitex
- Status changed from s1b_analysis_finished to s2a_design_started
comment:3 Changed 16 years ago by mitex
- Design_owners set to mitex
- Status changed from s2a_design_started to s3b_implementation_finished
- Imp._owners set to mitex
Design:
OpenBooksPalette.items: Instead of tracking, filtering and transforming list, use composing list that adds all open books and the desktop view. Use caching of already created items.
In BookItem.render() return "Show Desktop" if the item represents the desktop book.
Implementation:
[6872]
comment:4 Changed 16 years ago by mira
- Status changed from s3b_implementation_finished to s3c_implementation_ok
- Imp._score changed from 0 to 3.5
- Analysis_reviewers set to mitex
- Imp._reviewers set to mira
- The code for adding an BookItem was not reused for desctop book but copied. I extracted a common method for this.
- You should think of better way for caching BookItems (perhaps using an immutable for instance with good equals() and hashcode() methods)
merged at revision - [6972]
Note: See
TracTickets for help on using
tickets.