Skip to content

Commit 265e462

Browse files
committed
style: refine chip styling in headline tiles
- Use muted secondary container color - Reduce vertical padding on chip labels
1 parent a723299 commit 265e462

File tree

3 files changed

+14
-19
lines changed

3 files changed

+14
-19
lines changed

lib/shared/widgets/headline_tile_image_start.dart

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -166,12 +166,11 @@ class _HeadlineMetadataRow extends StatelessWidget {
166166
fontSize: baseTextStyle.fontSize != null ? baseTextStyle.fontSize! * 0.85 : 10, // 15% smaller or fallback
167167
);
168168
// Use the same derived style for chip labels for consistency
169-
final chipLabelStyle = metadataStyle;
169+
final chipLabelStyle = metadataStyle;
170170

171-
final chipBackgroundColor = colorScheme.surfaceContainerHighest.withOpacity(
172-
0.7,
173-
);
174-
const iconSize = AppSpacing.sm;
171+
// Use a muted version of the secondary container color for the chip background
172+
final chipBackgroundColor = colorScheme.secondaryContainer.withOpacity(0.4);
173+
const iconSize = AppSpacing.sm;
175174

176175
return Wrap(
177176
spacing: AppSpacing.xs,
@@ -213,8 +212,8 @@ class _HeadlineMetadataRow extends StatelessWidget {
213212
label: Text(headline.category!.name),
214213
labelStyle: chipLabelStyle,
215214
backgroundColor: chipBackgroundColor,
216-
padding: EdgeInsets.zero, // Minimal padding around label
217-
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2 ), // Reduced padding
215+
padding: EdgeInsets.zero,
216+
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2, vertical: 0), // Minimal vertical padding
218217
visualDensity: VisualDensity.compact,
219218
materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
220219
),
@@ -245,8 +244,8 @@ class _HeadlineMetadataRow extends StatelessWidget {
245244
label: Text(headline.source!.name),
246245
labelStyle: chipLabelStyle,
247246
backgroundColor: chipBackgroundColor,
248-
padding: EdgeInsets.zero, // Minimal padding around label
249-
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2), // Reduced padding
247+
padding: EdgeInsets.zero,
248+
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2, vertical: 0), // Minimal vertical padding
250249
visualDensity: VisualDensity.compact,
251250
materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
252251
),

lib/shared/widgets/headline_tile_image_top.dart

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -177,9 +177,7 @@ class _HeadlineMetadataRow extends StatelessWidget {
177177
);
178178
final chipLabelStyle = metadataStyle;
179179

180-
final chipBackgroundColor = colorScheme.surfaceContainerHighest.withOpacity(
181-
0.7,
182-
);
180+
final chipBackgroundColor = colorScheme.secondaryContainer.withOpacity(0.4);
183181
const iconSize = AppSpacing.sm;
184182

185183
return Wrap(
@@ -223,7 +221,7 @@ class _HeadlineMetadataRow extends StatelessWidget {
223221
labelStyle: chipLabelStyle,
224222
backgroundColor: chipBackgroundColor,
225223
padding: EdgeInsets.zero,
226-
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2 ),
224+
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2, vertical: 0),
227225
visualDensity: VisualDensity.compact,
228226
materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
229227
),
@@ -254,7 +252,7 @@ class _HeadlineMetadataRow extends StatelessWidget {
254252
labelStyle: chipLabelStyle,
255253
backgroundColor: chipBackgroundColor,
256254
padding: EdgeInsets.zero,
257-
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2),
255+
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2, vertical: 0),
258256
visualDensity: VisualDensity.compact,
259257
materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
260258
),

lib/shared/widgets/headline_tile_text_only.dart

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -126,9 +126,7 @@ class _HeadlineMetadataRow extends StatelessWidget {
126126
);
127127
final chipLabelStyle = metadataStyle;
128128

129-
final chipBackgroundColor = colorScheme.surfaceContainerHighest.withOpacity(
130-
0.7,
131-
);
129+
final chipBackgroundColor = colorScheme.secondaryContainer.withOpacity(0.4);
132130
const iconSize = AppSpacing.sm;
133131

134132
return Wrap(
@@ -172,7 +170,7 @@ class _HeadlineMetadataRow extends StatelessWidget {
172170
labelStyle: chipLabelStyle,
173171
backgroundColor: chipBackgroundColor,
174172
padding: EdgeInsets.zero,
175-
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2 ),
173+
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2, vertical: 0),
176174
visualDensity: VisualDensity.compact,
177175
materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
178176
),
@@ -203,7 +201,7 @@ class _HeadlineMetadataRow extends StatelessWidget {
203201
labelStyle: chipLabelStyle,
204202
backgroundColor: chipBackgroundColor,
205203
padding: EdgeInsets.zero,
206-
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2),
204+
labelPadding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs / 2, vertical: 0),
207205
visualDensity: VisualDensity.compact,
208206
materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
209207
),

0 commit comments

Comments
 (0)