trent_larson/crowd-funder-for-time-pwa
4
1
Fork 2
Code Issues Pull Requests 22 Releases 5 Wiki Activity

Refactor: unify dialog styles across components #181

Closed
jose wants to merge 0 commits from dialog-styles-unified into master
pull from: dialog-styles-unified
merge into: trent_larson:master
Conversation 0 Commits - Files Changed -
jose commented 2025-08-21 14:42:04 +00:00
Owner
Copy Link
  • Add unified .dialog-overlay and .dialog classes to tailwind.css
  • Remove duplicated dialog styles from 9 Vue components
  • Standardize z-index to 50 for all dialogs
  • Preserve special cases: FeedFilters (z-index: 100), PhotoDialog (camera styles)
  • Eliminate ~200 lines of duplicated CSS code
  • Updated ContactNameDialog from using getter methods to direct CSS classes
  • Improve maintainability with single source of truth for dialog styling
- Add unified .dialog-overlay and .dialog classes to tailwind.css - Remove duplicated dialog styles from 9 Vue components - Standardize z-index to 50 for all dialogs - Preserve special cases: FeedFilters (z-index: 100), PhotoDialog (camera styles) - Eliminate ~200 lines of duplicated CSS code - Updated ContactNameDialog from using getter methods to direct CSS classes - Improve maintainability with single source of truth for dialog styling
jose added 1 commit 2025-08-21 14:42:04 +00:00
Refactor: unify dialog styles across components 8f5111d100 
- Add unified .dialog-overlay and .dialog classes to tailwind.css
- Remove duplicated dialog styles from 9 Vue components
- Standardize z-index to 50 for all dialogs
- Preserve special cases: FeedFilters (z-index: 100), PhotoDialog (camera styles)
- Eliminate ~200 lines of duplicated CSS code
- Updated ContactNameDialog from using getter methods to direct CSS classes
- Improve maintainability with single source of truth for dialog styling
jose added a new dependency 2025-08-25 12:37:17 +00:00
#185 Fix: re-organized dialog and notification z-indices
jose removed a dependency 2025-08-26 06:42:22 +00:00
#185 Fix: re-organized dialog and notification z-indices
jose closed this pull request 2025-08-26 09:32:40 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
anomalist jose jsnbuchanan kentbull trentlarson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: trent_larson/crowd-funder-for-time-pwa#181
Delete Branch "dialog-styles-unified"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?