Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Access level for charts #119

Open
SnarfW opened this issue Aug 25, 2023 · 9 comments
Open

Access level for charts #119

SnarfW opened this issue Aug 25, 2023 · 9 comments

Comments

@SnarfW
Copy link

SnarfW commented Aug 25, 2023

I would like to hide the charts in the 'more charts menu' for visitors.
According to the instructions this should be possible via Control Panel > Charts
Unfortunately, the charts are always shown, even when I set the access level to 'Hide from everyone'. Is this a bug, or is something in my set-up hampering this to function correct?
I also see only one menu, I was expecting to see an option to set the access per chart.

afbeelding

I did now hide them all by setting the access level for the menu (Control Panel > Menu), which is working as expected

Using vesta 2.1.17.0.3
Webtrees 2.1.17

@ric2016
Copy link
Contributor

ric2016 commented Aug 25, 2023

These options are only for the original relationships chart.

The new charts are experimental and will be fully customizable as soon as webtrees supports container modules, see here .

Unfortunately there is no progress at all on this issue (and many others), and no indication whether this will change. It may help if you leave a comment for that issue, but I'm not optimistic that further custom module support is on the roadmap at all.

@SnarfW
Copy link
Author

SnarfW commented Aug 26, 2023

You mentioned that the new charts are experimental, I wasn't aware. They are include in your 'stable' release zip-file from your homepage. Wouldn't it be better not to include them (yet) in the zip?

I also see that there are many open issue for webtrees...

@ric2016
Copy link
Contributor

ric2016 commented Aug 28, 2023

Wouldn't it be better not to include them (yet) in the zip?

I optimistically was hoping that webtrees development would speed up again.
I still think it's ok to include them as the extra menu can be hidsden altogether, as you found out.
I may have to reconsider this if the webtrees development hiatus continues.

@kiwi3685
Copy link

As a temporary fix, to enable removing the "More charts" menu item while keeping the original relationships chart available, could you just add a new class to the menu item.

Like:
return new Menu(
I18N::translate('More Charts'),
'#',
'menu-chart more-charts',
['rel' => 'nofollow'],
$submenus->all());

Then "display:none; can be added in css by anyone who wants this option. Perhaps preferable than you removing these charts completely.

@ric2016
Copy link
Contributor

ric2016 commented Feb 21, 2024

Please support the container modules issue in webtrees to get this fixed properly.

You don't need css to hide the extra menu though: Just set its access level in the control panel to "hide from anyone". It may seem, due to the name, that this would affect the entire module, but it actually only affects the menu part.

@kiwi3685
Copy link

I trued that before I made the suggestion. It also removed the Relationship chart link from the main Charts menu group.
I can check it again tomorrow if you’re sure it should work.

@ric2016
Copy link
Contributor

ric2016 commented Feb 21, 2024

Yes, this works, I just re-checked myself. Make sure to set the access level under
Modules > Genealogy > Menus > Vesta Extended Relationships (and keep 'Enabled' checked).

@kiwi3685
Copy link

Thank you. Now I see the problem.

I was changing access to the item "Modules > Genealogy > Charts > Vesta Extended Relationships".

I should have known better.

@ric2016
Copy link
Contributor

ric2016 commented Feb 21, 2024

Well it is a bit confusing because the extra menu also contains charts.
It really would be much simpler with container modules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants