Closed rash805115 closed 1 week ago
The functionality is nice but the placement of the UI needs work. Adds too much space.
Hmm I agree. What would you suggest? Maybe keep the Page Options
div but make the actual options (next div) collapsible?
I was thinking if more options are ever added, they would be in line with the other options, so the height you see should not be more than what it is today.
Maybe in the sidebar somewhere? Not sure if possible.
That was my first try, in the right sidebar. But the UI was looking ugly, especially in mobile.
I think I made it to sidebar, and it looks better than before for sure.
Look on mobile,
Look on desktop,
Hi @milesj Can you take a look when you have a chance?
I'm out of town for the new few weeks.
Hello @milesj Just checking again if you are back in town, and if you can review this PR.
This PR is to add support for page options, the first one including
Hide Inherited
as explained in #150 .As shown in screenshot, at the top of the page you can see a new option - Page Options.
When unchecked (default), the page would show normally.
When checked, all attributes that were inherited, in sidebar and in page content, are hidden.