Summary: | Update 'Save Image' to more clear instructions | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Megan Gardner <megan_gardner> | ||||||||
Component: | New Bugs | Assignee: | Megan Gardner <megan_gardner> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, wenson_hsieh | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Megan Gardner
2019-04-11 15:28:30 PDT
Created attachment 367253 [details]
Patch
Comment on attachment 367253 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367253&action=review > Source/WebCore/en.lproj/Localizable.strings:629 > -/* Title for Save Image action button */ > -"Save Image" = "Save Image"; > +/* Title for Add to Photos action button */ > +"Add to Photos" = "Add to Photos"; I think these are in sorted order. IIRC, Tools/Scripts/update-webkit-localizable-strings should do the trick. > Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js:95 > +localizedStrings["Add to Photos"] = "Add to Photos"; Is this related? I think this is a separate web inspector feature... > Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js:951 > + contextMenu.appendItem(WI.UIString("Add to Photos"), () => { (Here too) Comment on attachment 367253 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367253&action=review >> Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js:95 >> +localizedStrings["Add to Photos"] = "Add to Photos"; > > Is this related? I think this is a separate web inspector feature... This is the feature Devin added in WI that allows us to snapshot a DOM node. We don't want to change this string. Created attachment 367254 [details]
Patch
Comment on attachment 367254 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367254&action=review > Source/WebInspectorUI/ChangeLog:15 > + Let's remove this ChangeLog entry. Created attachment 367258 [details]
Patch
Sorry, I meant to, I guess I missed it. Comment on attachment 367258 [details] Patch Clearing flags on attachment: 367258 Committed r244205: <https://trac.webkit.org/changeset/244205> |