mirror of
https://github.com/openai/codex.git
synced 2026-11-01 19:33:35 +00:00
Felipe Coury
de40696ec4
## Why Reading daily totals required selecting each bar, and changing grouping required opening a picker. Section navigation could shift vertically, while scroll hints appeared even when content fit. ## What changed - Make `g` cycle through available groupings and summary views directly. - Show daily totals on charts spanning up to seven days when space permits, prioritizing the selected value and keeping neighboring labels separate. - Keep section navigation at a stable height, show more legend rows in taller maximized reports, and move their update timestamp into the footer. - Hide scroll hints when content fits, including an exact fit, and show only the active summary view label. ## Testing Add regression tests and snapshots for seven-day totals, narrow label spacing, grouping wraparound and server capabilities, configured `g` navigation in ungrouped sections, stable tab placement, adaptive legends, footer timestamps, and overflow-dependent scroll hints. GitOrigin-RevId: 9d14f314b4a22ca2d926f0e69c6b89e4f9c8550d
76 lines
2.6 KiB
Rust
76 lines
2.6 KiB
Rust
//! Account profile loading and the Summary's year-view interaction.
|
|
//! Aggregation changes reuse the loaded profile; account refresh replaces all profile data.
|
|
|
|
use super::AnalyticsView;
|
|
use super::activity_chart::TokenActivityView;
|
|
use super::client::Live;
|
|
use super::data::Load;
|
|
use super::sections::Section;
|
|
use crate::keymap::ListAction;
|
|
use codex_backend_client::AccountProfile;
|
|
|
|
pub(super) const VIEWS: [TokenActivityView; 3] = [
|
|
TokenActivityView::Daily,
|
|
TokenActivityView::Weekly,
|
|
TokenActivityView::Cumulative,
|
|
];
|
|
|
|
impl Live {
|
|
pub(super) async fn profile(&self) -> Result<Option<AccountProfile>, String> {
|
|
let session = self.session().await?;
|
|
let profile = session
|
|
.backend
|
|
.request(|client| async move { client.get_account_profile().await })
|
|
.await;
|
|
session.backend.ensure_identity().await?;
|
|
profile.map(Some).map_err(super::client::request_error)
|
|
}
|
|
}
|
|
|
|
impl AnalyticsView {
|
|
pub(crate) fn select_summary(&mut self, view: Option<TokenActivityView>) {
|
|
self.section = Section::Summary;
|
|
self.zoomed = true;
|
|
self.scroll_offset = 0;
|
|
if let Some(view) = view {
|
|
self.sections[Section::Summary].group = VIEWS
|
|
.iter()
|
|
.position(|item| *item == view)
|
|
.unwrap_or_default();
|
|
}
|
|
}
|
|
|
|
pub(super) fn load_summary(&mut self) {
|
|
if let (Some((_, _, frame)), Some(live)) = (&self.connection, &self.live) {
|
|
let live = std::sync::Arc::clone(live);
|
|
self.profile = Load::start(async move { live.profile().await }, frame.clone());
|
|
}
|
|
}
|
|
|
|
pub(super) fn summary_action(&mut self, action: Option<ListAction>) {
|
|
let step = match action {
|
|
Some(ListAction::MoveUp) => Some(-1),
|
|
Some(ListAction::MoveDown) => Some(1),
|
|
Some(ListAction::PageUp) => Some(-(self.viewport_height as isize)),
|
|
Some(ListAction::PageDown) => Some(self.viewport_height as isize),
|
|
Some(ListAction::JumpTop) => {
|
|
self.scroll_offset = 0;
|
|
Some(0)
|
|
}
|
|
Some(ListAction::JumpBottom) => {
|
|
self.scroll_offset = usize::MAX;
|
|
Some(0)
|
|
}
|
|
Some(ListAction::Cancel) => {
|
|
self.is_done = true;
|
|
None
|
|
}
|
|
Some(ListAction::MoveLeft | ListAction::MoveRight | ListAction::Accept) | None => None,
|
|
};
|
|
if let Some(step) = step {
|
|
self.scroll_offset = self.scroll_offset.saturating_add_signed(step);
|
|
self.follow_selection = false;
|
|
}
|
|
}
|
|
}
|