Repository navigation
fix(layout): align select and text-field with Material 3 inline-flex standards - #152
Merged
Merged
Conversation
🚀 Deploying Preview to Cloudflare 🚀Preview URL: https://fix-select-full-width-material.treeder.workers.dev (commit 51c32a8)This URL reflects your latest Preview deploymentPreview Deployments by commit
|
There was a problem hiding this comment.
📋 Review Summary
This pull request updates the styling for md-select to set default full-width (width: 100%) on the host element and inner .wrapper, whilst also adding min-width: 0 on .field and .select. This aligns the layout behaviour of md-select with md-text-field across responsive forms and flex containers, preventing horizontal overflow when text content expands.
🔍 General Feedback
- Setting
width: 100%on:hostand.wrapperensures consistent sizing and behaviour betweenmd-selectandmd-text-fieldwhen placed inside form layouts and flex columns. - Applying
min-width: 0to.fieldand.selectadheres to CSS flexbox best practice, ensuring the elements can shrink properly inside flex containers without causing unexpected layout overflow. - The changes are clean, minimal, and follow existing styling patterns in the repository.
📊 Token Usage & Cost Efficiency
| Metric | Value |
|---|---|
| Input Tokens (uncached) | 31,192 |
| Input Tokens (cached) | 421,430 (⚡ 93.1% cached) |
| Thinking / Reasoning Tokens | 11,917 |
| Output Tokens | 234 |
| Total Session Tokens | 464,773 |
| Cost (uncached input) | $0.0234 |
| Cost (cached input) | $0.0316 |
| Cost (thinking / reasoning) | $0.0447 |
| Cost (response output) | $0.0009 |
| Estimated Total Cost | $0.1006 |
Gemini 3.8 Flash: introductory rate $0.75/$3.75 per 1M applied; reverts to $1.5/$7.5 after 2026-12-31.
Gemini 3.8 Flash: standard tier; batch and flex are half again, priority is higher.
Cache reads are priced here, but context-cache STORAGE, where the model charges for it separately per token-hour, is not included, so the figure can run slightly low.
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
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.
Summary
Align
<md-select>and<md-text-field>with Material Design 3 and Web Standards:• Form control host elements retain
display: inline-flexwith intrinsic width (no forcedwidth: 100%on:host), allowing natural inline/side-by-side usage.•
<md-select>inner.wrappernow expands towidth: 100%withmin-width: 0on.field, .select, ensuring the internal field container properly responds whenever the element is stretched by parent layouts (e.g. flex column forms) or explicitwidth: 100%.Key Changes
• select/select.js:
width: 100%on internal.wrapperso the field scales to the host width rather than staying clamped at 160px.min-width: 0on.field, .selectto prevent horizontal overflow in flex/grid layouts.:hostas standarddisplay: inline-flexwithout hardcodedwidth: 100%.• text/text-field.js:
width: 100%from:host, restoring standarddisplay: inline-flexbehavior matching<md-select>and Material 3 specifications.