Comment 7 for bug 627686

Revision history for this message
Nigel Jones (dev-nigelj) wrote :

Documenting discussion from IRC:

We decided that he best practice was to:

* Allow the undoing of the 'superexpand' - added a link (where X Opportunities Hidden would be seen) reading "Return to original results" - this copies the original results to a variable to be restored w/o additional queries
* When collapsing a package, also under the show all opportunities
* Change the X Opps Hidden text to: Show x hidden opportunities (Show hidden opportunity for 1 hidden)

I have also, as suggested:
* Fixed the non-javascript links (was already intended) & made the change to CSS to reformat/lowercase text
* Fixed the animations (i.e. turn off when loading is finished)

I think this is the best all-round solution for the issue.