154086 - Want /css to display current css setting

Status: RESOLVED FIXED

(Other Applications :: ChatZilla, enhancement)

Description

[[e-mail]]

Although /eval client.DEFAULT_STYLE will show you the current css setting I
think /css should as well as it is more intuitive.

Comment 1

[[e-mail]]

Attachment: Proposed patch

Comment 2

[Robert Ginda]

Comment on attachment 89048 [details] [diff] [review]
Proposed patch

Please update the .properties file to show that the parameter is optional, and
add a line to the help text that says what heppens when the parameter is not
provided.

After that, r=rginda.

Comment 3

[[e-mail]]

Attachment: Patch v1.1

Added []s to the usage to show that the parameter is optional.
Changed the help text from "Sets" to "Sets or displays".

Comment 4

[Robert Ginda]

Comment on attachment 89050 [details] [diff] [review]
Patch v1.1

r=rginda

Comment 5

[[e-mail]]

Fix was checked in by timeless.

Comment 6

[:deanis74]

-    if (!e.inputData)
-        return false;

Did we lose this false return value?

Comment 7

[Robert Ginda]

returning false from a command handler tells chatzilla that there was a problem
with the parameters, and that it should display usage information for the
command.  I don't think that's what we want here.

Comment 8

[:deanis74]

Ahh, I see now.  Makes perfect sense.