Opened 11 years ago

Closed 10 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 11 years ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 11 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

Added cachedView member and refactored source code with r4510.

comment:3 Changed 11 years ago by swagner

  • Owner changed from swagner to mkommend
  • Status changed from reviewing to assigned

comment:4 Changed 11 years ago by mkommend

  • Status changed from assigned to accepted

comment:5 Changed 11 years ago by mkommend

  • Status changed from accepted to reviewing

Corrected deregistering of event handlers in the view host with r4511.

comment:6 Changed 11 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 11 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 11 years ago by mkommend

  • Status changed from accepted to reviewing

comment:9 follow-up: Changed 11 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 11 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 11 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 10 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.1 to 3.3.2
Note: See TracTickets for help on using tickets.