Details
-
Type: Improvement
-
Status: Done/Fixed
-
Priority: Major
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.5
-
Fix Version/s: 4.6
-
Component/s: CiviReport
-
Labels:None
-
Documentation Required?:User and Admin Doc
Description
Pull in work done by Brian on http://dev.nysenate.gov/issues/6440.
Original Spec - http://wiki.civicrm.org/confluence/display/CRM/Report+UI+Improvements.
Some notes:
===========
- There are ~6 commits (excluding merges).
- Changes are mostly in report tpl files (of core) and css (additional css).
- Tabs have been created / structured in tpl file itself. Assuming we don't need a tab style like contribution/events (tabs created via php), this looks better, as there is no real need of firing a new ajax call for each of the tabs. The changes are quite clean & clear.
- It feels tempting to further structure the criteria.tpl so that all tabs have their own tpl. We might also want to do some renaming like - changing fields.tpl to configuration.tpl, instance.tpl to Configuration/settings.tpl ..
- Updating report has a confirmation process which looks useful and could be pulled in.
- We will need to make sure tabs display / hide works based on permissions. Looking at the code it seems it will work.
- Some reports have their own custom tpl and will require an update once we merge / improve the changes.