Skip to content

Tooltips!#1076

Open
R-Laine wants to merge 3 commits into
CyberTimon:mainfrom
R-Laine:Tool-tips
Open

Tooltips!#1076
R-Laine wants to merge 3 commits into
CyberTimon:mainfrom
R-Laine:Tool-tips

Conversation

@R-Laine
Copy link
Copy Markdown
Contributor

@R-Laine R-Laine commented Apr 23, 2026

Description

Added toggleable tooltips that appear on hover and take their text from a centralised file

Type of Change

  • New feature
  • UI/UX improvement

Changes Made

  • Added enableTooltips property to AppSettings interface in AppProperties.tsx
  • Added tooltips toggle switch in SettingsPanel with enable/disable functionality
  • Updated GlobalTooltip component to accept appSettings prop and respect the enableTooltips setting
  • Added data-tooltip prop to Slider component interface and implementation
  • Created tooltipText.ts utility file with centralized tooltip text for all adjustment sliders
  • Added tooltips to all Basic adjustment sliders (Brightness, Contrast, Highlights, Shadows, Whites, Blacks, Exposure)
  • Added tooltips to all Color adjustment sliders (Temperature, Tint, Vibrance, Saturation, Hue, Blending, Balance)
  • Added tooltips to all Details adjustment sliders (Sharpness, Clarity, Dehaze, Structure, Centré, Chromatic Aberration)
  • Added tooltips to all Effects adjustment sliders (Glow, Halation, Light Flares, Vignette controls, Grain controls)
  • Updated App.tsx to pass appSettings to GlobalTooltip component

Screenshots/Videos

image Screenshot 2026-04-23 225654

Testing

  • I have tested these changes locally and confirmed that they work as expected without issues

Test Configuration:

  • OS: Windows 11
  • Hardware: i9-9900 RX9070

Checklist

  • My code follows the project's code style
  • I haven't added unnecessary AI-generated code comments
  • My changes generate no new warnings or errors

AI Disclaimer:

Please state the involvement of AI in this PR:

  • This PR is AI-generated but guided by a human

@R-Laine R-Laine requested a review from CyberTimon as a code owner April 23, 2026 21:57
@R-Laine R-Laine closed this Apr 23, 2026
@R-Laine R-Laine deleted the Tool-tips branch April 23, 2026 22:40
@R-Laine R-Laine restored the Tool-tips branch April 23, 2026 23:12
@R-Laine R-Laine reopened this Apr 23, 2026
@CyberTimon
Copy link
Copy Markdown
Owner

Hi @R-Laine

Thanks for your PR and for the very clean implementation. I'm not sure if this is really needed after I've written the RapidRAW documentation: https://www.getrapidraw.com/docs/

What do you think?

@mmadesignerunknown
Copy link
Copy Markdown
Contributor

I think it's needed, not everyone will read the complete guide before starting you know.

@CyberTimon
Copy link
Copy Markdown
Owner

Thanks, I agree. Can we please rename the setting to something like Adjustments Tooltip which does additionally display your helpful tooltips over just adjustments and leaves the other tooltips unaffected? Because most people want tooltips on buttons which don't contain text but don't want tooltips over adjustments they use every day.

@Flohhhhh
Copy link
Copy Markdown
Contributor

Let's make sure we have clear and proper descriptions of brightness and exposure based on our discussion in #1105 !

@mmadesignerunknown
Copy link
Copy Markdown
Contributor

mmadesignerunknown commented Apr 29, 2026

Let's make sure we have clear and proper descriptions of brightness and exposure based on our discussion in #1105 !

Can you please share the image presenting tooltips here too?

@Flohhhhh
Copy link
Copy Markdown
Contributor

Flohhhhh commented Apr 29, 2026

Let's make sure we have clear and proper descriptions of brightness and exposure based on our discussion in #1105 !

Can you please share the image presenting tooltips here too?

Proposed examples for exposure/brightness tooltips:

• Exposure tooltip: "Linear EV change on RAW data. Like changing shutter/ISO. Can clip highlights/shadows."
• Brightness tooltip: "Filmic, midtone-focused brightness. Compresses highlights/shadows for smooth roll-off."

@mmadesignerunknown
Copy link
Copy Markdown
Contributor

Let's make sure we have clear and proper descriptions of brightness and exposure based on our discussion in #1105 !

Can you please share the image presenting tooltips here too?

Proposed examples for exposure/brightness tooltips:

• Exposure tooltip: "Linear EV change on RAW data. Like changing shutter/ISO. Can clip highlights/shadows."
• Brightness tooltip: "Filmic, midtone-focused brightness. Compresses highlights/shadows for smooth roll-off."

A short version here (I am fine with the long one too though):

• Exposure tooltip: "Adjusts light like camera exposure. Can clip highlights or shadows."
• Brightness tooltip: "Gentle brightness for midtones. Protects highlights and shadows."

@R-Laine
Copy link
Copy Markdown
Contributor Author

R-Laine commented Apr 30, 2026

Busy atm but if you give me a few days or someone takes over all of these can be done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants