Improve TranslationWizard navigation and UI help by NB-Core · Pull Request #40 · NB-Core/modules · GitHub
Skip to content

Improve TranslationWizard navigation and UI help#40

Merged
NB-Core merged 1 commit into
masterfrom
codex/revise-and-enhance-navigation-in-translation-wizard
Jul 14, 2025
Merged

Improve TranslationWizard navigation and UI help#40
NB-Core merged 1 commit into
masterfrom
codex/revise-and-enhance-navigation-in-translation-wizard

Conversation

@NB-Core

@NB-Core NB-Core commented Jul 14, 2025

Copy link
Copy Markdown
Owner

Summary

  • restructure Translation Wizard navigation into clear groups
  • add explanation of the wizard and navigation links to the start page

Testing

  • php -l systems/translationwizard/translationwizard/build_nav.php
  • php -l systems/translationwizard/translationwizard/default.php

https://chatgpt.com/codex/tasks/task_e_6874e2c941788329a6fe93b2c1c99d62

@NB-Core NB-Core requested a review from Copilot July 14, 2025 11:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the Translation Wizard by providing a clearer navigation structure and adding introductory help text on the start page.

  • Added welcome message and navigation instructions to the wizard’s default page
  • Grouped left-side navigation into “Information,” “Translation Tasks,” “Database Tools,” and “Settings”

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
systems/translationwizard/translationwizard/default.php Added welcome text and step-by-step navigation help
systems/translationwizard/translationwizard/build_nav.php Restructured navigation into labeled groups
Comments suppressed due to low confidence (2)

systems/translationwizard/translationwizard/build_nav.php:4

  • [nitpick] Indentation is inconsistent with other addnav calls; align this group heading to match surrounding style.
       addnav("Information");

systems/translationwizard/translationwizard/build_nav.php:8

  • [nitpick] Indentation is inconsistent with surrounding addnav calls; use the same indent level for group headings.
       addnav("Translation Tasks");

Comment thread systems/translationwizard/translationwizard/default.php
Comment thread systems/translationwizard/translationwizard/default.php
Comment thread systems/translationwizard/translationwizard/default.php
@NB-Core NB-Core merged commit b0f8218 into master Jul 14, 2025
@NB-Core NB-Core deleted the codex/revise-and-enhance-navigation-in-translation-wizard branch July 14, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants