Opened 16 years ago
Last modified 6 years ago
#7797 assigned enhancement
Consolidate iframe headers
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | 2.7 |
Component: | Administration | Keywords: | needs-patch |
Focuses: | Cc: |
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.
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.