plone / plone/mockup

Add `required` boolean option to Autotoc tabs

Open
#695 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

02 type: regression 05 type: question 41 lvl: easy 99 tag: Plone 4.3.x 99 tag: Plone 5.x 99 tag: UX Accessibility
Dominant language
JavaScript
Stars
57
Forks
103
Avg merge
10h 25m
Merged PRs (30d)
18

Description

The goal is feature parity with Plone 4.x, which by default displays a required icon in tabs that contain required fields.

It could be done by copying existing classes from level elements to tabs (.autotoc-nav a) -- and then updating the existing CSS selector, since pre-p.a.widgets one relies on an extra span.

However if we do it so generically, instead of required I wonder if we shouldn't we call this option classLevelCopyOrigin, copyOriginalLevelsClasses or something clearer?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the Autotoc tab implementation and the existing level-element classes referenced in the issue. Review how .autotoc-nav a and the current CSS selector handle required fields, then determine the option name and update the tab classes and selector so the required icon appears consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
css, javascript
Domain
frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.