Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

SJRK-435: fix ToC preview bug #104

Merged
merged 2 commits into from Dec 9, 2020
Merged

Conversation

BlueSlug
Copy link
Member

@BlueSlug BlueSlug commented Dec 7, 2020

UPDATE: This bug is solved by always hiding the UIO Table of Contents on the Edit page

Jira: https://issues.fluidproject.org/browse/SJRK-435

.flc-toc-tocContainer {
display: none !important;
}
</style>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than adding a <style> element, it may be cleared just to add an it as an inline style directly on toc container. It would also be good to leave a comment with a link to the JIRA to explain why it's hidden.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had tried this, since inline styling has the highest specificity, but turning the TOC preference on and off will make the element show again.

@jobara
Copy link
Member

jobara commented Dec 8, 2020

Rather than revert the commit, could you reset back and only include the change to the styling. It would also be good to update the JIRA with some information about approaches tried and the why the current solution was chosen. Just so we can keep a history of things.

@BlueSlug
Copy link
Member Author

BlueSlug commented Dec 8, 2020

I've added this comment to the Jira, detailing why the particular solution we chose is used and what other options were considered.

Ready for more review :)

@jobara jobara changed the title SJRK-435: make locationBar model relays one-way to fix ToC preview bug SJRK-435: fix ToC preview bug Dec 9, 2020
@jobara jobara merged commit c52e54e into fluid-project:main Dec 9, 2020
@jobara
Copy link
Member

jobara commented Dec 9, 2020

merged at c52e54e

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

Successfully merging this pull request may close these issues.

None yet

2 participants