A minor addendum to the bug fix I checked in a few minutes ago.

This commit is contained in:
hyatt%netscape.com 1999-08-12 21:20:04 +00:00
parent f8e268afd8
commit 53a1619128

View File

@ -884,24 +884,24 @@ nsMenuFrame::BuildAcceleratorText(nsString& aAccelString)
PRBool prependPlus = PR_FALSE;
if(commandValue != "") {
if(commandValue != "" && commandValue != "false") {
prependPlus = PR_TRUE;
aAccelString += "Ctrl"; // Hmmm. Kinda defeats the point of having an abstraction.
}
if(controlValue != "") {
if(controlValue != "" && controlValue != "false") {
prependPlus = PR_TRUE;
aAccelString += "Ctrl";
}
if(shiftValue != "") {
if(shiftValue != "" && shiftValue != "false") {
if (prependPlus)
aAccelString += "+";
prependPlus = PR_TRUE;
aAccelString += "Shift";
}
if (altValue != "") {
if (altValue != "" && altValue != "false") {
if (prependPlus)
aAccelString += "+";
prependPlus = PR_TRUE;