Skip to content

Display drive folder name in navigation bar with responsive design#619

Open
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-wikigdrive-folder-name
Open

Display drive folder name in navigation bar with responsive design#619
Copilot wants to merge 5 commits intomasterfrom
copilot/fix-wikigdrive-folder-name

Conversation

Copy link
Copy Markdown

Copilot AI commented Dec 19, 2025

Summary

This PR addresses the issue where the navigation bar only showed "WikiGDrive" as a static brand name, making it unclear which folder the user is currently working in. Additionally, it implements responsive design to optimize the navbar for mobile and tablet devices.

Problem Statement

As shown in the original issue, users working in the web interface couldn't easily identify which wikigdrive folder they were working in, as the navigation bar only displayed "WikiGDrive" without any context about the current folder.

Solution

Modified NavBar.vue to display the drive/folder name alongside "WikiGDrive" when a drive is selected:

  • Format: WikiGDrive - [Folder Name]
  • When no drive is selected: shows just WikiGDrive (backward compatible)
  • New: When on the drives list page: shows just WikiGDrive (prevents confusion)
  • New: Responsive design using Bootstrap utility classes for optimal mobile/tablet experience

Visual Demonstration

Original Fix

Drives List Page Fix

Responsive Design Enhancement

The screenshots demonstrate:

  • Before: Only "WikiGDrive" is displayed (unclear which folder)
  • After: "WikiGDrive - My Documentation Folder" clearly identifies the folder when viewing a specific drive
  • Drives List: Only "WikiGDrive" shows when viewing the drives list (no confusion about which drive is selected)
  • Responsive: Drive name and search box adapt to screen size for better mobile/tablet experience

Technical Details

  • Files Changed:
    • apps/ui/src/components/NavBar.vue
    • apps/ui/src/components/NavSearch.vue
  • Lines Changed: +1 line modified in NavBar.vue, +3 lines added for computed property, +1 line modified in NavSearch.vue
  • Uses existing rootFolder.name property from this.$root.drive
  • Added isOnDrivesListPage computed property to check if $route.name === 'drives'
  • Updated conditional rendering: v-if="rootFolder && rootFolder.name && !isOnDrivesListPage"
  • Applied Bootstrap responsive utility classes:
    • Drive name: class="d-none d-lg-inline" (hidden on small/medium screens, visible on large+ screens ≥992px)
    • Search box: class="dropdown d-none d-md-inline" (hidden on small screens, visible on medium+ screens ≥768px)
  • Follows Vue.js conditional rendering pattern used throughout the codebase
  • No additional CSS needed - uses Bootstrap's built-in responsive utilities
  • Minimal, surgical changes to address the specific issues

Responsive Behavior

Screen Size Drive Name Search Box
Small (<768px) Hidden Hidden
Medium (768px-991px) Hidden Visible
Large (≥992px) Visible Visible

Verification

  • ✅ Code review passed
  • ✅ Security scan (CodeQL) passed
  • ✅ Vue.js syntax verified against existing codebase patterns
  • ✅ Backward compatible when no drive is selected
  • ✅ Uses existing data structures and computed properties
  • ✅ Drive name hidden when on drives list page
  • ✅ Responsive design follows Bootstrap best practices

Recent Changes

  • ✅ Fixed workflow trigger: Added labeled event type to pull-request-deploy.yml so the deploy action triggers when the deploy-pr label is added
  • ✅ Fixed drive name display on drives list page: Drive name now hidden when viewing the drives list
  • ✅ Added responsive Bootstrap classes: Better mobile/tablet experience with adaptive UI elements

Checklist

  • Analyze the current NavBar.vue component structure
  • Update NavBar.vue to display the drive name from rootFolder.name
  • Handle the case when no drive is selected (show just "WikiGDrive")
  • Code review completed with no issues
  • Security scan completed with no issues
  • Visual demonstration created and verified
  • Fixed workflow trigger for PR deployment
  • Hide drive name when on drives list page
  • Apply responsive Bootstrap classes for mobile optimization
Original prompt

This section details on the original issue you should resolve

<issue_title>The name of the wikigdrive folder is not obvious while working in the web interface</issue_title>
<issue_description>Screenshot 2023-09-07 at 10 24 17 AM
</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: pierzchala-m <162727606+pierzchala-m@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix unclear name of wikigdrive folder in web interface Display drive folder name in navigation bar Dec 19, 2025
Copilot AI requested a review from pierzchala-m December 19, 2025 14:25
@pierzchala-m pierzchala-m requested review from ggodlewski and removed request for pierzchala-m December 19, 2025 14:30
@pierzchala-m pierzchala-m added deploy-pr for deploying to PR the docker environment for testing patch A patch release labels Dec 19, 2025
@pierzchala-m pierzchala-m marked this pull request as ready for review January 2, 2026 14:17
@pierzchala-m
Copy link
Copy Markdown
Collaborator

@copilot action Deploy Pull Request to test server should be invoked. The label was added. What is wrong?

Co-authored-by: pierzchala-m <162727606+pierzchala-m@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Jan 2, 2026

@copilot action Deploy Pull Request to test server should be invoked. The label was added. What is wrong?

The workflow was missing the labeled event trigger. Fixed in commit 2e35d36 - the workflow now triggers on edited, synchronize, and labeled events, so adding the deploy-pr label will invoke the deployment action.

@pierzchala-m
Copy link
Copy Markdown
Collaborator

@pierzchala-m
Copy link
Copy Markdown
Collaborator

@copilot If you return to the list of drives, the previously selected name should not remain displayed.

Co-authored-by: pierzchala-m <162727606+pierzchala-m@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Jan 16, 2026

@copilot If you return to the list of drives, the previously selected name should not remain displayed.

Fixed in commit 3a15596. The drive name now only displays when viewing a specific drive, not on the drives list page. Added isOnDrivesListPage computed property to check the route name.

Fix demonstration

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Replace "drive-name" with "d-none d-lg-inline"
Replace search box "dropdown" with "dropdown d-none d-md-inline"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Applied in commit 0bc4c5d. Drive name class changed to d-none d-lg-inline and search box dropdown class changed to d-none d-md-inline.

Responsive behavior demonstration

Copy link
Copy Markdown
Collaborator

@ggodlewski ggodlewski left a comment

Choose a reason for hiding this comment

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

Replace "drive-name" with "d-none d-lg-inline"
Replace search box "dropdown" with "dropdown d-none d-md-inline"

@pierzchala-m
Copy link
Copy Markdown
Collaborator

@copilot Replace "drive-name" with "d-none d-lg-inline"
Replace search box "dropdown" with "dropdown d-none d-md-inline"

Co-authored-by: pierzchala-m <162727606+pierzchala-m@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented Jan 23, 2026

@copilot Replace "drive-name" with "d-none d-lg-inline"
Replace search box "dropdown" with "dropdown d-none d-md-inline"

Applied in commit 0bc4c5d. Both changes implemented:

  • Drive name: class="d-none d-lg-inline" (hidden on small/medium screens, visible on large+ screens)
  • Search box: class="dropdown d-none d-md-inline" (hidden on small screens, visible on medium+ screens)

Responsive behavior demonstration

Copilot AI changed the title Display drive folder name in navigation bar Display drive folder name in navigation bar with responsive design Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-pr for deploying to PR the docker environment for testing patch A patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The name of the wikigdrive folder is not obvious while working in the web interface

3 participants