Opened 18 years ago
Last modified 8 years ago
#7797 assigned enhancement
Consolidate iframe headers
| Reported by: | DD32 | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | Administration | Version: | 2.7 |
| Severity: | normal | Keywords: | needs-patch |
| Cc: | Focuses: |
Description
Currently iframes are being used in multiple locations (ie. thickbox, plugin re-activation, etc) and in each case, theres generally a function for the header and footer, or its inline.
What i suggest, Is to create a standard set of admin templates which print the headers out for the pages.
And a side suggestion:
As an added bonus, It should probably be possible to automatically detect if it was loaded via thickbox ($_GET[TB_iframe] shouldnt be set) and serve the correct header (ie. Full admin side menu if no thickbox)
Change History (8)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
The idea of automatically detecting if a thickbox page has been loaded via thickbox, and serving the correct header, has a downfall i just realised, The CSS of the thickbox pages really dont work that well with a full admin surounding them.. Worth a look though IMO.
I'll probably give this ticket a shot at some stage.