Skip to content

Improve: Move directory name into General tab in Directory Builder#2709

Open
RabbiIslamRony wants to merge 1 commit intodevelopmentfrom
improve/move-directory-name-to-general-tab
Open

Improve: Move directory name into General tab in Directory Builder#2709
RabbiIslamRony wants to merge 1 commit intodevelopmentfrom
improve/move-directory-name-to-general-tab

Conversation

@RabbiIslamRony
Copy link
Member

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

Moves the editable directory name from the page header into the General tab of the Directory Builder as a proper form field. The header now shows the name as a read-only label. https://prnt.sc/oGc1_GertBzV

Why: The old inline-editable name in the header looked like a static page title — many users never discovered they could click it to rename. Placing it inside the General tab with a label, description, and standard text input makes it immediately obvious and consistent with every other setting.

How to test

  1. Go to Directory Builder → Edit any directory type (e.g. "Business").
  2. Confirm the General tab now shows a "Directory name" section as the first field with the current name pre-filled.
  3. Change the name in the input, click Update, and verify:
    • The header label updates to reflect the new name.
    • The directory is saved with the new name (check the term in the database or revisit the page).
  4. Confirm the header name text is non-interactive (no pointer cursor, no click-to-edit).
  5. Create a new directory type and verify the name field appears empty with placeholder text.

Any linked issues

Fixes #

Checklist

@RabbiIslamRony
Copy link
Member Author

@rafizsejim Suggested by @devShamim — moved the directory name into the General tab for better discoverability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments