Repository navigation
Conversation
Clarify that Phoenix Pro does not charge the displayed AI cost, and explain Claude or ChatGPT subscription coverage and API-provider billing. Style the compact cost note with an info icon, underline, and hover/focus details. Give every chart day a full-height hover target and outline while preserving the independently scaled usage fill. Pin Phoenix Pro to 38fd8bb41efac31a5503a375d0e37eba57426d81 for the matching tooltip behavior, chart markup, integration coverage, and tracking updates. Include the pending Node lockfile version alignment with 5.5.9. Verified in the native Linux app with 31/31 Usage Calendar and 13/13 Usage Meter integration checks. Targeted lint, LESS compilation, normal-app layout and hit testing, and whitespace checks passed.
Allow 280px for the spend explanation so the no-charge sentence fits on one line. Keep normal wrapping and cap the width to the viewport. Pin Pro to ca5eadcea57e781b515af6c811aeaa99ab69e81a for the companion fix that prevents the usage chip's native title from appearing in its popup. Verified the sentence occupies one line and the tooltip stays within the Windows viewport. LESS compiles; styles are synced and refreshed on Mac and Windows, and whitespace checks pass.
Remove the shared upward checkbox offset and adjust the top margin so the checkmarks align with the Connect and Enable AI headings. Remove the nested path hint's bottom margin to give its divider equal spacing. Pin Pro to dc29bbb624ebae9b3f1e9c1496a61075764b9045 for the matching path hint placement and tab-specific divider behavior. Verified both settings tabs directly on Linux, macOS, and Windows without changing input values. LESS compilation, targeted lint, and whitespace checks passed. No unit or integration tests were run, as requested.
|
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.
Clarify that Phoenix Pro does not charge the displayed AI cost, and explain Claude or ChatGPT subscription coverage and API-provider billing.
Style the compact cost note with an info icon, underline, and hover/focus details. Give every chart day a full-height hover target and outline while preserving the independently scaled usage fill.
Pin Phoenix Pro to 38fd8bb41efac31a5503a375d0e37eba57426d81 for the matching tooltip behavior, chart markup, integration coverage, and tracking updates. Include the pending Node lockfile version alignment with 5.5.9.
Verified in the native Linux app with 31/31 Usage Calendar and 13/13 Usage Meter integration checks. Targeted lint, LESS compilation, normal-app layout and hit testing, and whitespace checks passed.