Hi
2016-10-10 17:29 GMT+02:00 Vincent Massol <vincent(a)massol.net>et>:
Hi Guillaume,
On 10 Oct 2016, at 16:43, Guillaume Delhumeau
<
guillaume.delhumeau(a)xwiki.com> wrote:
Hello.
In Flamingo, we have a variable called $displayPageHeader, defined in
layoutvars.vm.
When it is set, the logo is not displayed in the top menu, but on a
header
under it. It is used by some clients, where they
could also add some
extra
content.
hmmm… I’m wondering if this is the best way.
We already a UIXP for page header so if some users want a custom header
they can already do so. I don’t think it belongs to the theme.
Indeed, adding content by modifying the header is not the best way.
Anyway, I think it is still useful to have this option. In the header, the
user have the ability to display a bigger logo for example, and a
background corresponding to its visual identity. And it's easier than
creating a UIX for this.
Since Caty has created the issue
http://jira.xwiki.org/browse/XWIKI-13160,
maybe she has some example to show us.
So instead I would have a boolean to decide whether to display the logo or
not. Actually even that boolean could be avoided by deciding that if the
logo field is empty then no logo is displayed.
When no logo is chosen, the current behavior is to fallback to the XWiki
default one.
Am I missing something?
Thanks
-Vincent
For your information, the code I was working on is here:
https://github.com/xwiki/xwiki-platform/compare/feature-theme-editor?expand…
and ready to be merged.
Thanks,
Guillaume
Example:
http://tof.canardpc.com/view/83040d2d-79c4-4f1b-ab1b-af8ee5f3be62.jpg
I'd like to expose this variable into the Flamingo Theme, see:
http://jira.xwiki.org/secure/attachment/33074/33074_preview.png
However, in order to enable the live preview in the theme editor, I need
to
be able to change the value of the
$displayPageHeader variable on the
fly,
or at least by setting a parameter in the query
string.
This variable will be:
displayPageHeader
Possible values:
"true", "false"
Default value:
Fallback to the Flamingo Theme, "false" if empty.
I don't have other use-case to cover with this variable.
Here is my +1.
Thanks,
--
Guillaume Delhumeau (guillaume.delhumeau(a)xwiki.com)
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Guillaume Delhumeau (guillaume.delhumeau(a)xwiki.com)
Research & Development Engineer at XWiki SAS
Committer on the
XWiki.org project