Permalink control: Implement methods activate/deactivate.

[jorix]

The attached patch implements the methods activate/deactivate on the control.

The patch includes a specific test.

The tests work well in: FF36, IE8, Chrome6 & Safari5.

[ahocevar]

Why would someone want to deactivate and reactivate a Permalink control? See #2519. The patch adds a lot of code which is not very useful in my opinion (think library size).

[jorix]

Replying to ahocevar:

Why would someone want to deactivate and reactivate a Permalink control? See #2519.

Only to clean the scene of the map. My concern is that now activate/deactivate do nothing, but in others controls it act. In what must act and which are not?

The patch adds a lot of code which is not very useful in my opinion (think library size).

Yes in 2.x, much of the additional code is to keep the parameter "element" of the constructor.

[ahocevar]

As a rule of thumb, controls that do not create Handler instances do not support activate/deactivate.

[jorix]

Replying to ahocevar:

As a rule of thumb, controls that do not create Handler instances do not support activate/deactivate.

Then Panel", "Graticule" and "MousePosition" are exceptions?, hence my confusion comes the #2520 and #2567 tickets were approved.

[ahocevar]

Whatever. Sorry if my closing the ticket was biased. Reopening and setting state back to review.

[jorix]

"Needs more work" for the same reason that #2822

[jorix]

Closed as "wontfix" for the same reason that #2822