Bug 291 - Nieprawidłowy accesskey dla Ponów w menedżerze zakładek
Summary: Nieprawidłowy accesskey dla Ponów w menedżerze zakładek
Status: VERIFIED FIXED
Alias: None
Product: Firefox
Classification: Tłumaczenia oprogramowania
Component: Ogólne (show other bugs)
Version: nieokreślona
Hardware: Wszystkie Wszystkie
: P2 trywialny
Assignee: Marek Stępień
QA Contact: Staś Małolepszy :stas
URL:
Depends on:
Blocks: 391
  Show dependency treegraph
 
Reported: 2004-11-07 01:47 CET by Marek Stępień
Modified: 2009-01-02 03:23 CET (History)
3 users (show)

See Also:


Attachments
patch (trunk) (2.32 KB, patch)
2005-05-19 12:43 CEST, Marek Stępień
marcoos+bugs: review-
Details
Patch (trunk) (2.72 KB, patch)
2005-05-19 23:18 CEST, Marek Stępień
no flags Details

Description Marek Stępień 2004-11-07 01:47:56 CET
To jest spinoff błędu 270. Jedyny fragment, który nie wszedł do 1.0, bo nie
moglismy dojsc, w ktorym to jest miejscu w zrodlach.

Fragment opisu z bledu 270:

--
Okno "Menedżer zakładek"
Menu "Edycja"
Pokazuje się pozycja "Ponów(R)"

Na przykład, jeśli przeciągniemy zakładkę do innego katalogu, to następnie w
"Edycji" otrzymamy pozycje:
- Cofnij przeciąganie zakładkę (trzeba to również poprawić - "przeciąganie
zakładki")
- Ponów(R)
Ponów przeciąganie zakładki jest OK
--

Poprawka w nastepnej wersji po 1.0.
Comment 1 Marek Stępień 2005-03-18 18:28:08 CET
Masówka. Likwidacja produktu "Firefox pl-PL", błędy zostają przeniesione do
Lokalizacja/Firefox.
Comment 2 Marek Stępień 2005-04-16 23:40:26 CEST
Ktoś wie, gdzie to jest (w sensie *.dtd/*.properties)? Bo ja nie mogę znaleźć...
Comment 4 Staś Małolepszy :stas 2005-04-17 18:46:47 CEST
Chyba nie.. tam wszędzie dla redo jest accesskey "n". Podejrzana sprawa.
Comment 5 Zbigniew Braniecki 2005-04-17 18:55:34 CEST
W wersji trunk 1.1 mi dziala Przywroc ma accesskey "r".
Comment 6 Marek Stępień 2005-04-18 00:25:38 CEST
Zbudowałeś to z tymi localami?
Comment 7 Marek Stępień 2005-04-18 00:27:24 CEST
Nie "przywróć". _Ponów_.

Nie zrozumiałeś. :P

Reop.
Comment 8 Zbigniew Braniecki 2005-04-18 00:28:45 CEST
tak, zbudowalem.
Comment 9 Marek Stępień 2005-05-19 12:43:07 CEST
Created attachment 459 [details]
patch (trunk)

Problem polegał na tym, że w dtd było "Przywróć" i accesskey "r",
natomiast w properties używaliśmy "Ponów $czynność". 

Rozwiązanie są dwa:
1) Zamienić na Ponów w *.dtd i ustalić nowy accesskey (to rozwiązanie
byłoby bardziej zgodne z terminologią Windows i Gnome, ale dla mnie słowo
"Ponów" jest w tym kontekście nieprawidłowe, niezależnie od tego, co MS i
GnomePL zrobiły ;))
2) Zamienić na Przywróć w *.properties (to rozwiązanie jest prostsze i
spójniejsze z resztą Firefoksa - w głównym oknie mamy Przywróć)

Załączony patch stosuje rozwiązanie 2.
Comment 10 Marek Stępień 2005-05-19 23:11:34 CEST
Comment on attachment 459 [details]
patch (trunk)

r=smalolepszy, ew. gandalf.
Comment 11 Marek Stępień 2005-05-19 23:14:21 CEST
Comment on attachment 459 [details]
patch (trunk)

Nieważne. Patch ma błąd.
Comment 12 Marek Stępień 2005-05-19 23:18:32 CEST
Created attachment 460 [details]
Patch (trunk)

Lepszy patch.

Te same zmiano co poprzednio plus zmiana:
cmd_*_redo_accesskey = n
na
cmd_*_redo_accesskey = r

Właściwie tylko dla porządku, bo wygląda na to, że accesskey i tak jest
brany z innego pliku, tu można by wpisać byle co, a i tak accesskeyem byłoby
"r"
Comment 13 Marek Stępień 2005-05-19 23:19:07 CEST
Comment on attachment 460 [details]
Patch (trunk)

review=smalolepszy? ew. gandalf.

Teraz IMHO jest ok.
Comment 14 Marek Stępień 2005-05-21 20:36:54 CEST
Target -> DP alpha 1

(przepraszam za bugspam, maile można odfiltrować po haśle: "jeleniowypark1")
Comment 15 Marek Stępień 2005-05-24 01:16:36 CEST
Wrzucone do CVSa jakoś przedwczoraj.

Nikt się nie zainteresował ewentualnym review, więc na własne ryzyko to wrzuciłem.

FIXED.

Comment 16 Zbigniew Braniecki 2005-05-24 16:15:59 CEST
Hmmm... Marku, jesli prosisz o review i go nie ma, to nie jest najlepszym
wyjsciem zignorowac to i pucic patch bez review. Mozna zmienic requesta na kogos
innego, albo sprobowac sie dowiedziec czemu Staszyna nie dal Ci r+.

Note You need to log in before you can comment on or make changes to this bug.