openedx / openedx/frontend-app-authoring
Sidebar "Edit" button visual enhancement
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 17
- Forks
- 218
- Avg merge
- 9d 20h
- Merged PRs (30d)
- 20
Description
Designs
The current and proposed sidebar header actions are shown below:
| Current Designs | Updated Designs |
|---|---|
This story relates to the Edit/Open button. Publish button changes are addressed in Publish button should reflect publish status #1794.
| Button State Designs | Description |
|---|---|
| Edit: For component blocks | |
| Open: For containers, Units, Subsections, Sections |
Spec
- Edit button should appear in the sidebar header actions row for components.
- Edit button should have the edit pencil icon
- Open button should appear in the sidebar header actions for Units, Subsections, and Sections
- Open button should have the expand icon
- Edit and open buttons should be sized to hug text to give weight to the publish button
Not in scope
- Overflow menu behavior
- Publish button changes
Related Tickets
Publish button should reflect publish status #1794
Publish section/subsection #1977
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the sidebar header actions and the existing rendering paths for components, Units, Subsections, and Sections. Compare them with the linked designs, then verify that the appropriate Edit or Open icon and text-hugging size appear for each specified item while overflow and Publish behavior remain unchanged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- design, frontend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100