Clonemental - Overriding elements/header.php

Permalink
I have a client who wants to completely remove the <header> section containing the navigation and search box.

I created a new file application/elements/header.php containing just this:
<?php  defined('C5_EXECUTE') or die('Access Denied.');
$this->inc('elements/header_top.php');
and it shows in the overrides section of the environment dashboard, but has no effect.

Cache has been cleared/disabled during development.

Do overrides not work for packaged themes, or am I doing this in the wrong place?

How do you make modifications that won't be overwritten if a package gets updated?

jasteele12
View Replies:
hutman replied on at Permalink Reply
hutman
Overriding a single file from a packaged theme is not easy like that unfortunately.

And yes, anytime you upgrade that package any changes you make to the files will get wiped out.

No advice on how to do this "correctly" though.
jasteele12 replied on at Permalink Reply
jasteele12
I also tried it in application/themes/clonemental/elements/header.php but it didn't work there either.

This seems like a major step backwards from 5.6 :(

What, am I supposed to tell a client "Don't ever update that package!" ???
mnakalay replied on at Permalink Reply
mnakalay
Hello,
I'm a bit surprised. I didn't use clonemental but with elemental (so not a package) when having the file application/themes/elemental/elements/header.php it does override the original one.

One thing if you're using a package is to try to use $view->inc('filename') instead of $this->inc('filename'). That might work.

Alternatively, you could forget about the whole overriding issue and use my free Elemental Cloner packagehttps://www.concrete5.org/marketplace/addons/elemental-cloner...

It will let you make a clone of Elemental as a new packaged and independent theme. Like that you can modify it directly without worrying about overrides or future updates.
jasteele12 replied on at Permalink Reply
jasteele12
While both were listed in environment overrides, neither worked :(

This feels like a bug...

Thanks, I'll take a look at your package!