mirror of
https://github.com/NickeManarin/ScreenToGif.git
synced 2026-04-25 15:15:51 +03:00
[PR #1205] [MERGED] Fix "Mouse Events" button tooltip #1331
Labels
No labels
copy cats
duplicated
future feature
pull-request
⬜ Accepted
⬜ Completed
⬜ Help Wanted 💪
⬜ In Progress
⬜ Missing Details
⬜ Pending
⬜ Waiting For Answer ⏳
🆕 feature preview
🔷 Bug 🐛
🔷 Out Of Scope
🔷 Out Of Scope
🔷 Question
🔷Enhancement
🔷Enhancement
🔷Invalid / External
🔷Knowledge Base
🔷Won't Fix
🕑 High
🕑 High
🕑 High
🕕 Medium
🕙 Low
🕛 Critical
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ScreenToGif#1331
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/NickeManarin/ScreenToGif/pull/1205
Author: @matthew-e-brown
Created: 6/14/2023
Status: ✅ Merged
Merged: 6/23/2023
Merged by: @NickeManarin
Base:
dev← Head:localization-fix📝 Commits (4)
47e2af3Update StringResources.ru.xamle866fd7Replace all '.MouseClicks' with '.MouseEvents'9c58525Update EN 'S.Command.MouseEvents' tooltip textee7dd3cUpdate FR 'S.Command.MouseEvents' tooltip text📊 Changes
27 files changed (+46 additions, -46 deletions)
View changed files
📝
ScreenToGif/Resources/Localization/StringResources.ar.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.cs.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.da.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.de.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.el.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.en.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.es-AR.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.es.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.fi.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.fr.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.he.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.hu.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.it.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.ja.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.ko.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.nl.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.pl.xaml(+1 -1)📝
ScreenToGif/Resources/Localization/StringResources.pt-PT.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.pt.xaml(+2 -2)📝
ScreenToGif/Resources/Localization/StringResources.ru.xaml(+3 -3)...and 7 more files
📄 Description
PR #1046 changed the "Mouse Clicks" panel/button to "Mouse Events" instead, since mouse-related events could now be visible without clicking; a sensible change. However, the PR missed a few references to
*.MouseClicksin the code, resulting in the tooltip for the button not showing up correctly.This PR switches all references to
*.MouseClicksto their respective*.MouseEventsversions in all language.xamlfiles in/Resources/Localization. Additionally, it updates the values for the text itself from "Mouse clicks" to "Mouse events" within the tooltip for English and French (because those are the two languages that I know).It may be a good idea to let those on the localization teams know that "Mouse clicks" should be changed to "Mouse events" for consistency. I also noticed several references in the other files to
S.MouseClicks.Warning.None—this string no longer appears, since it's not really possible for a recording not to have any generic "mouse events" like it is for clicks. It can probably be removed from the language files. I don't think either of these last two points are particularly urgent, though.This PR fixes #1200.
Screenshots
Before
After
Cheers,
-- Matt
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.