Compare commits

..

2 Commits

Author SHA1 Message Date
38e65619e9 Fix tooltips clipped by overflow containers, improve Backend tooltip text
All checks were successful
Build App / compute-version (push) Successful in 4s
Build App / build-macos (push) Successful in 2m21s
Build App / build-windows (push) Successful in 3m28s
Build App / build-linux (push) Successful in 5m44s
Build App / create-tag (push) Successful in 6s
Build App / sync-to-github (push) Successful in 12s
Rewrite Tooltip to use React portal (createPortal to document.body) so
tooltips render above all UI elements regardless of ancestor overflow:hidden.
Also increased max-width from 220px to 280px for longer descriptions.

Expanded Backend tooltip to explain each option (Anthropic, Bedrock,
Ollama, LiteLLM) with practical context for new users.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-12 09:56:50 -07:00
d2c1c2108a Fix update checker to use full semver comparison and correct platform filtering
Some checks failed
Build App / compute-version (push) Successful in 5s
Build App / build-macos (push) Successful in 2m20s
Build App / build-windows (push) Successful in 3m28s
Build App / build-linux (push) Successful in 5m21s
Build App / create-tag (push) Successful in 3s
Build App / sync-to-github (push) Has been cancelled
The version comparison was only comparing the patch number, ignoring major
and minor versions. This meant 0.1.75 (patch=75) appeared "newer" than
0.2.1 (patch=1), and updates within 0.2.x were missed entirely.

Also fixed platform filtering to handle -mac suffix (previously only
filtered -win, so Linux users would see macOS releases too).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-12 09:45:43 -07:00
3 changed files with 77 additions and 72 deletions

View File

@@ -34,30 +34,37 @@ pub async fn check_for_updates() -> Result<Option<UpdateInfo>, String> {
.map_err(|e| format!("Failed to parse releases: {}", e))?; .map_err(|e| format!("Failed to parse releases: {}", e))?;
let current_version = env!("CARGO_PKG_VERSION"); let current_version = env!("CARGO_PKG_VERSION");
let is_windows = cfg!(target_os = "windows"); let current_semver = parse_semver(current_version).unwrap_or((0, 0, 0));
// Determine platform suffix for tag filtering
let platform_suffix: &str = if cfg!(target_os = "windows") {
"-win"
} else if cfg!(target_os = "macos") {
"-mac"
} else {
"" // Linux uses bare tags (no suffix)
};
// Filter releases by platform tag suffix // Filter releases by platform tag suffix
let platform_releases: Vec<&GiteaRelease> = releases let platform_releases: Vec<&GiteaRelease> = releases
.iter() .iter()
.filter(|r| { .filter(|r| {
if is_windows { if platform_suffix.is_empty() {
r.tag_name.ends_with("-win") // Linux: bare tag only (no -win, no -mac)
!r.tag_name.ends_with("-win") && !r.tag_name.ends_with("-mac")
} else { } else {
!r.tag_name.ends_with("-win") r.tag_name.ends_with(platform_suffix)
} }
}) })
.collect(); .collect();
// Find the latest release with a higher patch version // Find the latest release with a higher semver version
// Version format: 0.1.X or v0.1.X (tag may have prefix/suffix) let mut best: Option<(&GiteaRelease, (u32, u32, u32))> = None;
let current_patch = parse_patch_version(current_version).unwrap_or(0);
let mut best: Option<(&GiteaRelease, u32)> = None;
for release in &platform_releases { for release in &platform_releases {
if let Some(patch) = parse_patch_from_tag(&release.tag_name) { if let Some(ver) = parse_semver_from_tag(&release.tag_name) {
if patch > current_patch { if ver > current_semver {
if best.is_none() || patch > best.unwrap().1 { if best.is_none() || ver > best.unwrap().1 {
best = Some((release, patch)); best = Some((release, ver));
} }
} }
} }
@@ -92,36 +99,34 @@ pub async fn check_for_updates() -> Result<Option<UpdateInfo>, String> {
} }
} }
/// Parse patch version from a semver string like "0.1.5" -> 5 /// Parse a semver string like "0.2.5" -> (0, 2, 5)
fn parse_patch_version(version: &str) -> Option<u32> { fn parse_semver(version: &str) -> Option<(u32, u32, u32)> {
let clean = version.trim_start_matches('v'); let clean = version.trim_start_matches('v');
let parts: Vec<&str> = clean.split('.').collect(); let parts: Vec<&str> = clean.split('.').collect();
if parts.len() >= 3 { if parts.len() >= 3 {
parts[2].parse().ok() let major = parts[0].parse().ok()?;
let minor = parts[1].parse().ok()?;
let patch = parts[2].parse().ok()?;
Some((major, minor, patch))
} else { } else {
None None
} }
} }
/// Parse patch version from a tag like "v0.1.5", "v0.1.5-win", "0.1.5" -> 5 /// Parse semver from a tag like "v0.2.5", "v0.2.5-win", "v0.2.5-mac" -> (0, 2, 5)
fn parse_patch_from_tag(tag: &str) -> Option<u32> { fn parse_semver_from_tag(tag: &str) -> Option<(u32, u32, u32)> {
let clean = tag.trim_start_matches('v'); let clean = tag.trim_start_matches('v');
// Remove platform suffix // Remove platform suffix
let clean = clean.strip_suffix("-win").unwrap_or(clean); let clean = clean.strip_suffix("-win")
parse_patch_version(clean) .or_else(|| clean.strip_suffix("-mac"))
.unwrap_or(clean);
parse_semver(clean)
} }
/// Extract a clean version string from a tag like "v0.1.5-win" -> "0.1.5" /// Extract a clean version string from a tag like "v0.2.5-win" -> "0.2.5"
fn extract_version_from_tag(tag: &str) -> Option<String> { fn extract_version_from_tag(tag: &str) -> Option<String> {
let clean = tag.trim_start_matches('v'); let (major, minor, patch) = parse_semver_from_tag(tag)?;
let clean = clean.strip_suffix("-win").unwrap_or(clean); Some(format!("{}.{}.{}", major, minor, patch))
// Validate it looks like a version
let parts: Vec<&str> = clean.split('.').collect();
if parts.len() >= 3 && parts.iter().all(|p| p.parse::<u32>().is_ok()) {
Some(clean.to_string())
} else {
None
}
} }
/// Check whether a newer container image is available in the registry. /// Check whether a newer container image is available in the registry.

View File

@@ -449,7 +449,7 @@ export default function ProjectCard({ project }: Props) {
<div className="mt-2 ml-4 space-y-2 min-w-0 overflow-hidden"> <div className="mt-2 ml-4 space-y-2 min-w-0 overflow-hidden">
{/* Backend selector */} {/* Backend selector */}
<div className="flex items-center gap-1 text-xs"> <div className="flex items-center gap-1 text-xs">
<span className="text-[var(--text-secondary)] mr-1">Backend:<Tooltip text="Anthropic = direct Claude API via OAuth. Bedrock = AWS Bedrock. Ollama = local models. LiteLLM = proxy gateway for 100+ providers." /></span> <span className="text-[var(--text-secondary)] mr-1">Backend:<Tooltip text="Choose the AI model provider for this project. Anthropic: Connect directly to Claude via OAuth login (run 'claude login' in terminal). Bedrock: Route through AWS Bedrock using your AWS credentials. Ollama: Use locally-hosted open-source models (Llama, Mistral, etc.) via an Ollama server. LiteLLM: Connect through a LiteLLM proxy gateway to access 100+ model providers (OpenAI, Azure, Gemini, etc.)." /></span>
<select <select
value={project.backend} value={project.backend}
onChange={(e) => { e.stopPropagation(); handleBackendChange(e.target.value as Backend); }} onChange={(e) => { e.stopPropagation(); handleBackendChange(e.target.value as Backend); }}

View File

@@ -1,4 +1,5 @@
import { useState, useRef, useEffect, type ReactNode } from "react"; import { useState, useRef, useLayoutEffect, type ReactNode } from "react";
import { createPortal } from "react-dom";
interface TooltipProps { interface TooltipProps {
text: string; text: string;
@@ -7,53 +8,44 @@ interface TooltipProps {
/** /**
* A small circled question-mark icon that shows a tooltip on hover. * A small circled question-mark icon that shows a tooltip on hover.
* Renders inline and automatically repositions to stay within the viewport. * Uses a portal to render at `document.body` so the tooltip is never
* clipped by ancestor `overflow: hidden` containers.
*/ */
export default function Tooltip({ text, children }: TooltipProps) { export default function Tooltip({ text, children }: TooltipProps) {
const [visible, setVisible] = useState(false); const [visible, setVisible] = useState(false);
const [position, setPosition] = useState<"top" | "bottom">("top"); const [coords, setCoords] = useState({ top: 0, left: 0 });
const [align, setAlign] = useState<"center" | "left" | "right">("center"); const [, setPlacement] = useState<"top" | "bottom">("top");
const triggerRef = useRef<HTMLSpanElement>(null); const triggerRef = useRef<HTMLSpanElement>(null);
const tooltipRef = useRef<HTMLDivElement>(null); const tooltipRef = useRef<HTMLDivElement>(null);
useEffect(() => { useLayoutEffect(() => {
if (!visible || !triggerRef.current || !tooltipRef.current) return; if (!visible || !triggerRef.current || !tooltipRef.current) return;
const triggerRect = triggerRef.current.getBoundingClientRect(); const trigger = triggerRef.current.getBoundingClientRect();
const tooltipRect = tooltipRef.current.getBoundingClientRect(); const tooltip = tooltipRef.current.getBoundingClientRect();
const gap = 6;
// Decide vertical position: prefer top, fall back to bottom // Vertical: prefer above, fall back to below
if (triggerRect.top - tooltipRect.height - 6 < 4) { const above = trigger.top - tooltip.height - gap >= 4;
setPosition("bottom"); const pos = above ? "top" : "bottom";
} else { setPlacement(pos);
setPosition("top");
}
// Decide horizontal alignment const top =
const centerLeft = triggerRect.left + triggerRect.width / 2 - tooltipRect.width / 2; pos === "top"
const centerRight = centerLeft + tooltipRect.width; ? trigger.top - tooltip.height - gap
if (centerLeft < 4) { : trigger.bottom + gap;
setAlign("left");
} else if (centerRight > window.innerWidth - 4) { // Horizontal: center on trigger, clamp to viewport
setAlign("right"); let left = trigger.left + trigger.width / 2 - tooltip.width / 2;
} else { left = Math.max(4, Math.min(left, window.innerWidth - tooltip.width - 4));
setAlign("center");
} setCoords({ top, left });
}, [visible]); }, [visible]);
const positionClasses = position === "top" ? "bottom-full mb-1.5" : "top-full mt-1.5";
const alignClasses =
align === "left"
? "left-0"
: align === "right"
? "right-0"
: "left-1/2 -translate-x-1/2";
return ( return (
<span <span
ref={triggerRef} ref={triggerRef}
className="relative inline-flex items-center ml-1" className="inline-flex items-center ml-1"
onMouseEnter={() => setVisible(true)} onMouseEnter={() => setVisible(true)}
onMouseLeave={() => setVisible(false)} onMouseLeave={() => setVisible(false)}
> >
@@ -65,13 +57,21 @@ export default function Tooltip({ text, children }: TooltipProps) {
? ?
</span> </span>
)} )}
{visible && ( {visible &&
createPortal(
<div <div
ref={tooltipRef} ref={tooltipRef}
className={`absolute z-50 ${positionClasses} ${alignClasses} px-2.5 py-1.5 text-[11px] leading-snug text-[var(--text-primary)] bg-[var(--bg-tertiary)] border border-[var(--border-color)] rounded shadow-lg whitespace-normal max-w-[220px] w-max pointer-events-none`} style={{
position: "fixed",
top: coords.top,
left: coords.left,
zIndex: 9999,
}}
className={`px-2.5 py-1.5 text-[11px] leading-snug text-[var(--text-primary)] bg-[var(--bg-tertiary)] border border-[var(--border-color)] rounded shadow-lg whitespace-normal max-w-[280px] w-max pointer-events-none`}
> >
{text} {text}
</div> </div>,
document.body
)} )}
</span> </span>
); );