mai
/
ponshu-room-lite
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 1 Wiki Activity

Step 1 Cleanup: Safe code quality improvements 

- Replace deprecated withOpacity() with withValues(alpha:) across 13 files
- Remove unnecessary imports (schema imports, unused dart:io, etc.)
- Reduce analysis issues from 122 to 111 (11 issues resolved)
- All changes are safe and do not break functionality

Changes:
- lib/main.dart: Remove redundant schema imports
- lib/screens/*.dart: Update color methods, clean imports
- lib/widgets/*.dart: Update color methods
- lib/theme/app_theme.dart: Update color methods

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Ponshu Developer 2026-01-12 00:42:39 +09:00
parent 2eb451296b
commit a6cefb8be5
16 changed files with 46 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
.claude/settings.local.json
View File

@ -2,7 +2,15 @@
"permissions": {
"allow": [
"Bash(dir libsecrets.dart)",
"Bash(flutter analyze:*)"
"Bash(flutter analyze:*)",
"Bash(git remote add:*)",
"Bash(git branch:*)",
"Bash(git push:*)",
"Bash(git reset:*)",
"Bash(find:*)",
"Bash(git add:*)",
"Bash(git commit:*)",
"Bash(git rebase:*)"
],
"deny": [],
"ask": []

1
lib/main.dart
View File

@ -7,7 +7,6 @@ import 'models/user_profile.dart';
import 'models/menu_settings.dart';
import 'providers/theme_provider.dart';
import 'screens/main_screen.dart';
import 'services/migration_service.dart';
void main() async {

6
lib/screens/camera_screen.dart
View File

@ -468,7 +468,7 @@ class _CameraScreenState extends ConsumerState<CameraScreen> with SingleTickerPr
height: 180,
width: 4, // Thin track
decoration: BoxDecoration(
color: Colors.white.withOpacity(0.3),
color: Colors.white.withValues(alpha: 0.3),
borderRadius: BorderRadius.circular(2),
),
child: Stack(
@ -542,9 +542,9 @@ class _CameraScreenState extends ConsumerState<CameraScreen> with SingleTickerPr
Container(
padding: const EdgeInsets.symmetric(horizontal: 8, vertical: 6),
decoration: BoxDecoration(
color: Colors.black.withOpacity(0.6),
color: Colors.black.withValues(alpha: 0.6),
borderRadius: BorderRadius.circular(20),
border: Border.all(color: Colors.white.withOpacity(0.3), width: 1),
border: Border.all(color: Colors.white.withValues(alpha: 0.3), width: 1),
),
child: Row(
mainAxisSize: MainAxisSize.min,

2
lib/screens/home_screen.dart
View File

@ -1,4 +1,3 @@
// CupertinoPicker
import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';
import '../providers/theme_provider.dart';
@ -6,7 +5,6 @@ import '../providers/display_mode_provider.dart';
import 'camera_screen.dart';
import 'menu_creation_screen.dart';
import '../theme/app_theme.dart';
import '../providers/sake_list_provider.dart';
import '../providers/filter_providers.dart';
import '../providers/menu_providers.dart'; // Phase 2-1

6
lib/screens/menu_creation_screen.dart
View File

@ -89,9 +89,9 @@ class MenuCreationScreen extends ConsumerWidget {
return Container(
padding: const EdgeInsets.all(12),
decoration: BoxDecoration(
color: Colors.orange.withOpacity(0.1),
color: Colors.orange.withValues(alpha: 0.1),
border: Border(
bottom: BorderSide(color: Colors.orange.withOpacity(0.3)),
bottom: BorderSide(color: Colors.orange.withValues(alpha: 0.3)),
),
),
child: Row(
@ -172,7 +172,7 @@ class MenuCreationScreen extends ConsumerWidget {
color: Theme.of(context).cardColor,
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.05),
color: Colors.black.withValues(alpha: 0.05),
blurRadius: 8,
offset: const Offset(0, -2),
),

14
lib/screens/menu_pricing_screen.dart
View File

@ -137,9 +137,9 @@ class _MenuPricingScreenState extends ConsumerState<MenuPricingScreen> {
Container(
padding: const EdgeInsets.all(12),
decoration: BoxDecoration(
color: Colors.blue.withOpacity(0.1),
color: Colors.blue.withValues(alpha: 0.1),
border: Border(
bottom: BorderSide(color: Colors.blue.withOpacity(0.3)),
bottom: BorderSide(color: Colors.blue.withValues(alpha: 0.3)),
),
),
child: Row(
@ -213,7 +213,7 @@ class _MenuPricingScreenState extends ConsumerState<MenuPricingScreen> {
color: Theme.of(context).cardColor,
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.05),
color: Colors.black.withValues(alpha: 0.05),
blurRadius: 8,
offset: const Offset(0, -2),
),
@ -347,9 +347,9 @@ class _MenuPricingScreenState extends ConsumerState<MenuPricingScreen> {
Container(
padding: const EdgeInsets.all(12),
decoration: BoxDecoration(
color: Colors.grey.withOpacity(0.05),
color: Colors.grey.withValues(alpha: 0.05),
borderRadius: BorderRadius.circular(8),
border: Border.all(color: Colors.grey.withOpacity(0.2)),
border: Border.all(color: Colors.grey.withValues(alpha: 0.2)),
),
child: variants.isEmpty
? Row(
@ -394,9 +394,9 @@ class _MenuPricingScreenState extends ConsumerState<MenuPricingScreen> {
Container(
padding: const EdgeInsets.all(12),
decoration: BoxDecoration(
color: Colors.red.withOpacity(0.05),
color: Colors.red.withValues(alpha: 0.05),
borderRadius: BorderRadius.circular(8),
border: Border.all(color: Colors.red.withOpacity(0.3)),
border: Border.all(color: Colors.red.withValues(alpha: 0.3)),
),
child: Row(
mainAxisAlignment: MainAxisAlignment.center,

6
lib/screens/menu_settings_screen.dart
View File

@ -227,7 +227,7 @@ class _MenuSettingsScreenState extends ConsumerState<MenuSettingsScreen> {
Card(
elevation: 0,
color: Theme.of(context).colorScheme.surfaceContainerHighest.withOpacity(0.3),
color: Theme.of(context).colorScheme.surfaceContainerHighest.withValues(alpha: 0.3),
shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(12)),
child: Column(
children: [
@ -404,7 +404,7 @@ class _MenuSettingsScreenState extends ConsumerState<MenuSettingsScreen> {
color: Theme.of(context).cardColor,
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.05),
color: Colors.black.withValues(alpha: 0.05),
blurRadius: 8,
offset: const Offset(0, -2),
),
@ -521,7 +521,7 @@ class _MenuSettingsScreenState extends ConsumerState<MenuSettingsScreen> {
padding: const EdgeInsets.symmetric(vertical: 16),
decoration: BoxDecoration(
color: isSelected
? colorScheme.primaryContainer.withOpacity(0.2)
? colorScheme.primaryContainer.withValues(alpha: 0.2)
: colorScheme.surfaceContainerHighest,
borderRadius: BorderRadius.circular(12),
border: Border.all(

4
lib/screens/pdf_preview_screen.dart
View File

@ -104,7 +104,7 @@ class PdfPreviewScreen extends ConsumerWidget {
child: Container(
padding: const EdgeInsets.symmetric(horizontal: 12, vertical: 8),
decoration: BoxDecoration(
color: Colors.black.withOpacity(0.7),
color: Colors.black.withValues(alpha: 0.7),
borderRadius: BorderRadius.circular(8),
),
child: Row(
@ -131,7 +131,7 @@ class PdfPreviewScreen extends ConsumerWidget {
color: Theme.of(context).cardColor,
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.05),
color: Colors.black.withValues(alpha: 0.05),
blurRadius: 8,
offset: const Offset(0, -2),
),

8
lib/screens/placeholders/brewery_map_screen.dart
View File

@ -122,7 +122,7 @@ class _BreweryMapScreenState extends ConsumerState<BreweryMapScreen> {
right: 16,
child: FloatingActionButton.small(
heroTag: 'map_reset',
backgroundColor: Colors.white.withOpacity(0.9),
backgroundColor: Colors.white.withValues(alpha: 0.9),
foregroundColor: AppTheme.posimaiBlue,
elevation: 2,
onPressed: () {
@ -190,7 +190,7 @@ class _BreweryMapScreenState extends ConsumerState<BreweryMapScreen> {
borderRadius: BorderRadius.circular(16),
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.05),
color: Colors.black.withValues(alpha: 0.05),
blurRadius: 10,
offset: const Offset(0, 4),
),
@ -330,7 +330,7 @@ class _BreweryMapScreenState extends ConsumerState<BreweryMapScreen> {
Widget _buildRegionCard(BuildContext context, String name, int current, int total, bool isComplete, VoidCallback onTap) {
final color = isComplete ? AppTheme.posimaiBlue : Theme.of(context).cardColor;
final textColor = isComplete ? Colors.white : Theme.of(context).textTheme.bodyLarge?.color;
final subTextColor = isComplete ? Colors.white.withOpacity(0.8) : Colors.grey[600];
final subTextColor = isComplete ? Colors.white.withValues(alpha: 0.8) : Colors.grey[600];
return InkWell(
onTap: onTap,
@ -342,7 +342,7 @@ class _BreweryMapScreenState extends ConsumerState<BreweryMapScreen> {
borderRadius: BorderRadius.circular(12),
border: isComplete ? null : Border.all(color: Colors.grey[200]!), // Lighter border
boxShadow: [
if(!isComplete) BoxShadow(color: Colors.black.withOpacity(0.03), blurRadius: 4, offset: const Offset(0,2))
if(!isComplete) BoxShadow(color: Colors.black.withValues(alpha: 0.03), blurRadius: 4, offset: const Offset(0,2))
]
),
child: Column( // changed to column for 3-grid layout

8
lib/screens/placeholders/sommelier_screen.dart
View File

@ -109,7 +109,7 @@ class _SommelierScreenState extends ConsumerState<SommelierScreen> {
),
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.1),
color: Colors.black.withValues(alpha: 0.1),
blurRadius: 20,
offset: const Offset(0, 10),
),
@ -124,7 +124,7 @@ class _SommelierScreenState extends ConsumerState<SommelierScreen> {
child: Icon(
LucideIcons.sparkles,
size: 150,
color: isDark ? Colors.white.withOpacity(0.05) : Colors.blue.withOpacity(0.05),
color: isDark ? Colors.white.withValues(alpha: 0.05) : Colors.blue.withValues(alpha: 0.05),
),
),
@ -162,7 +162,7 @@ class _SommelierScreenState extends ConsumerState<SommelierScreen> {
color: AppTheme.posimaiBlue,
shadows: [
Shadow(
color: AppTheme.posimaiBlue.withOpacity(0.3),
color: AppTheme.posimaiBlue.withValues(alpha: 0.3),
blurRadius: 10,
offset: const Offset(0, 2),
),
@ -199,7 +199,7 @@ class _SommelierScreenState extends ConsumerState<SommelierScreen> {
Container(
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 8),
decoration: BoxDecoration(
color: Theme.of(context).colorScheme.surface.withOpacity(0.5),
color: Theme.of(context).colorScheme.surface.withValues(alpha: 0.5),
borderRadius: BorderRadius.circular(12),
),
child: Text(

2
lib/screens/sake_detail_screen.dart
View File

@ -1386,7 +1386,7 @@ class _PhotoEditModalState extends State<_PhotoEditModal> {
color: Theme.of(context).cardColor,
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.05),
color: Colors.black.withValues(alpha: 0.05),
blurRadius: 8,
offset: const Offset(0, -2),
),

12
lib/screens/scan_screen.dart
View File

@ -73,8 +73,8 @@ class _ScanARScreenState extends State<ScanARScreen> with SingleTickerProviderSt
Container(
decoration: BoxDecoration(
border: Border.symmetric(
horizontal: BorderSide(color: Colors.black.withOpacity(0.5), width: 100),
vertical: BorderSide(color: Colors.black.withOpacity(0.5), width: 40),
horizontal: BorderSide(color: Colors.black.withValues(alpha: 0.5), width: 100),
vertical: BorderSide(color: Colors.black.withValues(alpha: 0.5), width: 40),
),
),
),
@ -83,11 +83,11 @@ class _ScanARScreenState extends State<ScanARScreen> with SingleTickerProviderSt
width: 280,
height: 280,
decoration: BoxDecoration(
border: Border.all(color: AppTheme.posimaiBlue.withOpacity(0.8), width: 2),
border: Border.all(color: AppTheme.posimaiBlue.withValues(alpha: 0.8), width: 2),
borderRadius: BorderRadius.circular(16),
boxShadow: [
BoxShadow(
color: AppTheme.posimaiBlue.withOpacity(0.4),
color: AppTheme.posimaiBlue.withValues(alpha: 0.4),
blurRadius: 20,
spreadRadius: 2,
),
@ -104,7 +104,7 @@ class _ScanARScreenState extends State<ScanARScreen> with SingleTickerProviderSt
Center(
child: Container(
height: 2,
color: Colors.white.withOpacity(0.5),
color: Colors.white.withValues(alpha: 0.5),
),
)
],
@ -227,7 +227,7 @@ class _DigitalSakeCardDialog extends StatelessWidget {
),
borderRadius: BorderRadius.circular(20),
boxShadow: [
BoxShadow(color: Colors.black.withOpacity(0.2), blurRadius: 20, spreadRadius: 5),
BoxShadow(color: Colors.black.withValues(alpha: 0.2), blurRadius: 20, spreadRadius: 5),
],
border: Border.all(color: Colors.white, width: 2),
),

2
lib/theme/app_theme.dart
View File

@ -81,7 +81,7 @@ class AppTheme {
backgroundColor: (brightness == Brightness.dark) ? const Color(0xFF1E1E1E) : null,
indicatorColor: (brightness == Brightness.dark)
? const Color(0xFF376495)
: const Color(0xFF376495).withOpacity(0.25),
: const Color(0xFF376495).withValues(alpha: 0.25),
// Removed custom MaterialStateProperty text styles to avoid potential type issues for now.
// Default text style usually adapts to brightness if colorScheme.onSurface/onPrimary is set correctly.
),

2
lib/widgets/home/sake_grid_item.dart
View File

@ -162,7 +162,7 @@ class SakeGridItem extends ConsumerWidget {
shape: BoxShape.circle,
boxShadow: [
BoxShadow(
color: Colors.black.withOpacity(0.2),
color: Colors.black.withValues(alpha: 0.2),
blurRadius: 4,
offset: const Offset(0, 2),
),

2
lib/widgets/home/sake_list_item.dart
View File

@ -26,7 +26,7 @@ class SakeListItem extends ConsumerWidget {
final isDark = Theme.of(context).brightness == Brightness.dark;
// Adaptive selection color
final selectedColor = isDark ? Colors.orange.withOpacity(0.3) : Colors.orange.shade50;
final selectedColor = isDark ? Colors.orange.withValues(alpha: 0.3) : Colors.orange.shade50;
return Card(
clipBehavior: Clip.antiAlias,

2
lib/widgets/prefecture_filter_sheet.dart
View File

@ -69,7 +69,7 @@ class PrefectureFilterSheet extends ConsumerWidget {
Container(
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
decoration: BoxDecoration(
border: Border(bottom: BorderSide(color: Colors.grey.withOpacity(0.2))),
border: Border(bottom: BorderSide(color: Colors.grey.withValues(alpha: 0.2))),
),
child: Row(
mainAxisAlignment: MainAxisAlignment.spaceBetween,