[iOS] Support multiple file selection in UIDocumentPickerViewController
rdar://problem/49963514
Created attachment 367634 [details] Patch
Comment on attachment 367634 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367634&action=review > Source/WebCore/en.lproj/Localizable.strings:68 > +/* label next to file upload control; parameter is the number of files when greater than one */ > +"%lu files" = "%lu files"; Why is the string up two lines not good enough? :D Or replace that one with this one? I don't know what types you need.
Comment on attachment 367634 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367634&action=review >> Source/WebCore/en.lproj/Localizable.strings:68 >> +"%lu files" = "%lu files"; > > Why is the string up two lines not good enough? :D Or replace that one with this one? I don't know what types you need. lol, will use the other one via the handy multipleFileUploadText() in LocalizedStrings.h :) Thanks for reviewing!
Created attachment 367669 [details] Patch
Comment on attachment 367669 [details] Patch Clearing flags on attachment: 367669 Committed r244404: <https://trac.webkit.org/changeset/244404>
All reviewed patches have been landed. Closing bug.
> > Source/WebCore/en.lproj/Localizable.strings:68 > > +/* label next to file upload control; parameter is the number of files when greater than one */ > > +"%lu files" = "%lu files"; > > Why is the string up two lines not good enough? :D Or replace that one with > this one? I don't know what types you need. It actually isn't good at all! Pluralization doesn't work like "1 vs. any other number" in many languages, see <http://www.unicode.org/cldr/charts/latest/supplemental/language_plural_rules.html>. Foundation has proper support for this. I checked the Russian localization in shipping WebKit for macOS, and it actually uses the hack that translators apply when software lacks full localization support - it's translated to an equivalent of "Files: %d".
(In reply to Alexey Proskuryakov from comment #8) > > > Source/WebCore/en.lproj/Localizable.strings:68 > > > +/* label next to file upload control; parameter is the number of files when greater than one */ > > > +"%lu files" = "%lu files"; > > > > Why is the string up two lines not good enough? :D Or replace that one with > > this one? I don't know what types you need. > > It actually isn't good at all! > > Pluralization doesn't work like "1 vs. any other number" in many languages, > see > <http://www.unicode.org/cldr/charts/latest/supplemental/ > language_plural_rules.html>. Foundation has proper support for this. > > I checked the Russian localization in shipping WebKit for macOS, and it > actually uses the hack that translators apply when software lacks full > localization support - it's translated to an equivalent of "Files: %d". I was going to comment about this, but I don't think WebKit has a stringsdict yet? And didn't want to force bringing that up on Andy :)
Created bug for it https://bugs.webkit.org/show_bug.cgi?id=197070
> I was going to comment about this, but I don't think WebKit has a > stringsdict yet? And didn't want to force bringing that up on Andy :) Actually it does, Source/WebCore/en.lproj/Localizable.stringsdict. Since all localization is done through WebCore, that's the one to use.