Conversation
jankapunkt
added
question
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request optimizes the application's mobile user interface by implementing responsive design patterns across multiple pages and components. The changes focus on improving navigation, fixing dialog responsiveness issues, and ensuring consistent mobile experiences throughout the application.
- Migrated from
CreateDialog.vueto a newFormDialog.vuecomponent with trigger slot pattern for better reusability - Applied responsive Tailwind classes (mobile-first with
md:breakpoints) across editors, pages, and UI components - Enhanced mobile navigation with improved styling, animations, and close button functionality
Reviewed Changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
web/resources/js/dialogs/FormDialog.vue |
New dialog component replacing CreateDialog with trigger slot pattern and improved state management |
web/resources/js/dialogs/DialogBase.vue |
Added close button support, adjusted z-index, improved mobile responsiveness and backdrop opacity |
web/resources/js/dialogs/CreateDialog.vue |
Removed old dialog implementation in favor of FormDialog |
web/resources/js/dialogs/useCreateDialog.js |
Marked composable as deprecated and added close function |
web/resources/js/editor/PreparationsEditor.vue |
Applied responsive padding adjustments for toolbar and status bar |
web/resources/js/editor/EditorToolbar.vue |
Changed layout from inline-flex to block on mobile with lg breakpoint |
web/resources/js/Pages/coding/CodingEditor.vue |
Responsive zoom controls and status bar positioning |
web/resources/js/Pages/CodingPage.vue |
Migrated to FormDialog with trigger slot pattern |
web/resources/js/Pages/Projects/ProjectsListMenu.vue |
Integrated FormDialog and moved initSearch to onMounted |
web/resources/js/Pages/Projects/codebooks/ProjectCodebooks.vue |
Applied responsive spacing and button widths |
web/resources/js/Pages/ProjectsList.vue |
Simplified by removing CreateDialog integration |
web/resources/js/Pages/ProjectOverview.vue |
Removed CreateDialog references |
web/resources/js/Pages/PreparationPage.vue |
Fixed overflow and responsive button widths |
web/resources/js/Pages/coding/tree/SelectionList.vue |
Added horizontal scroll for long selection text on mobile |
web/resources/js/Pages/coding/tree/CodeTreeItemRenderer.vue |
Increased button padding on mobile |
web/resources/js/Pages/Teams/Partials/TeamMemberManager.vue |
Applied responsive layout for user list items |
web/resources/js/Pages/Profile/Partials/LegalForm.vue |
Made legal sections responsive with block layout on mobile |
web/resources/js/Layouts/AuthenticatedLayout.vue |
Redesigned mobile sidebar with improved navigation, removed redundant elements |
web/resources/js/Components/wizard/WizardOptions.vue |
Added change handler and data-index for select options |
web/resources/js/Components/lists/ResponsiveTabList.vue |
Adjusted tab spacing for mobile displays |
web/resources/js/Components/layout/SideOverlay.vue |
Reduced max-width on mobile devices |
web/resources/js/Components/global/Audit.vue |
Applied responsive spacing to filters |
web/resources/js/Components/files/FilesManager.vue |
Migrated to FormDialog with responsive button widths |
web/resources/js/Components/files/FilesList.vue |
Enhanced menu button styling and made table responsive |
web/plugins/visualization/src/list/ListView.vue |
Added horizontal scroll for selection text |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
kadewe
approved these changes
Nov 22, 2025
Contributor
kadewe
left a comment
kadewe
left a comment
There was a problem hiding this comment.
Quite a lot of work done here to make it more mobile friendly!
I like the new FormDialog.vue component – this saves a lot of redundant frontend code!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request fixes/optimizes navigation, functions and pages for mobile devices.