Opened 14 years ago
Closed 14 years ago
#1211 closed enhancement (done)
The ViewHost should cache the last displayed view if the content is set to null
| Reported by: | mkommend | Owned by: | mkommend |
|---|---|---|---|
| Priority: | medium | Milestone: | HeuristicLab 3.3.2 |
| Component: | MainForm.WindowsForms | Version: | 3.3.2 |
| Keywords: | Cc: |
Description
This ticket is related to #1204 and enables collection views the reuse of the detail view.
Change History (12)
comment:1 Changed 14 years ago by mkommend
- Status changed from new to accepted
comment:2 Changed 14 years ago by mkommend
- Owner changed from mkommend to swagner
- Status changed from accepted to reviewing
comment:3 Changed 14 years ago by swagner
- Owner changed from swagner to mkommend
- Status changed from reviewing to assigned
comment:4 Changed 14 years ago by mkommend
- Status changed from assigned to accepted
comment:5 Changed 14 years ago by mkommend
- Status changed from accepted to reviewing
Corrected deregistering of event handlers in the view host with r4511.
comment:6 Changed 14 years ago by mkommend
- Status changed from reviewing to assigned
The content events are not deregistered on disposed views which causes a memory leak.
comment:7 Changed 14 years ago by mkommend
- Status changed from assigned to accepted
Deregistered content events before a view is disposed in the ViewHost with r4521.
comment:8 Changed 14 years ago by mkommend
- Status changed from accepted to reviewing
comment:9 follow-up: ↓ 10 Changed 14 years ago by mkommend
In a discussion with swagner it was decided that the whole caching mechanism should be implemented instead of only caching the last displayed view.
comment:10 in reply to: ↑ 9 Changed 14 years ago by swagner
Replying to mkommend:
In a discussion with swagner it was decided that the whole caching mechanism should be implemented instead of only caching the last displayed view.
Really? Sorry, but I do not remember. I thought, we rather wanted to get rid of cached views? Please let us discuss this again ...
comment:11 Changed 14 years ago by mkommend
- Status changed from reviewing to readytorelease
This ticket will be closed and every further discussion / implementation on the ViewHost must be described in a separate ticket.
comment:12 Changed 14 years ago by swagner
- Resolution set to done
- Status changed from readytorelease to closed
- Version changed from 3.3.1 to 3.3.2
Added cachedView member and refactored source code with r4510.