Beefy Boxes and Bandwidth Generously Provided by pair Networks
Welcome to the Monastery
 
PerlMonks  

Re^3: New Behavior for Signatures?

by Chady (Priest)
on Jun 03, 2004 at 14:07 UTC ( #360153=note: print w/ replies, xml ) Need Help??


in reply to Re^2: New Behavior for Signatures?
in thread New Behavior for Signatures?

I see,

I had a quick glance at add_signature; it issues its warning without looking for a op=preview param.

If the param is set, then it should return the warning, else, it should assume that the display is not a preview and thus not show the warning. But I guess it is implemented that way to warn people who do not preview their nodes.


He who asks will be a fool for five minutes, but he who doesn't ask will remain a fool for life.

Chady | http://chady.net/


Comment on Re^3: New Behavior for Signatures?
Re^4: New Behavior for Signatures? (is bug)
by tye (Cardinal) on Jun 03, 2004 at 23:26 UTC

    I consider it a bug, but not a serious one. Checking for op=preview may not be the correct solution because it wouldn't catch problems introduced right before "create" -- but I'm not sure such are caught at all anyway. But this warning should not appear before content is editted.

    The solution I would use is to fill in a default template when it is empty but the sig isn't emply. This default template should end in a couple of newlines. The start of it should be small, perhaps "<!-- Type here -->". Someone volunteered to try to do this long ago, but I don't ever expect schedules from volunteers so I'm not surprised nor disappointed that it hasn't been done.

    - tye        

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://360153]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others romping around the Monastery: (6)
As of 2014-10-21 05:27 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    For retirement, I am banking on:










    Results (96 votes), past polls