Playing around with the 'frontend_comment' hook (eg emoticonchooser) at
https://github.com/s9y/Serendipity/blob ... rm.tpl#L38
I thought it might be better to move the hook right above or beneath the textarea, since we have serendipity_comment_s9ymarkup and serendipity_comment_emoticate Note-Blocks inbetween.
What do you think?
2k11: commentform hook
2k11: commentform hook
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Re: 2k11: commentform hook
I think it deviates too much from well-known behaviour for a proposed default template.
If said deviation would be considered acceptable, I'd maybe want to move it in front of the whole comment form. My UX point of view: once I started filling out a comment form, I want to fill it out and send it, so any further info is just a distraction.
YL
If said deviation would be considered acceptable, I'd maybe want to move it in front of the whole comment form. My UX point of view: once I started filling out a comment form, I want to fill it out and send it, so any further info is just a distraction.
YL
Re: 2k11: commentform hook
Well, in the backend textareas it is above since ages. That is a well known behaviour too.
I know that differs a little from commentforms... but (btw, what is a UX point? )
here in the forum it is on the right side, and would also be not nice having 2 or 3 blocks of text beween the textarea and a quick (mouse) sidejump.
I know that differs a little from commentforms... but (btw, what is a UX point? )
here in the forum it is on the right side, and would also be not nice having 2 or 3 blocks of text beween the textarea and a quick (mouse) sidejump.
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Re: 2k11: commentform hook
“UX point of view” – UX is user experience, usability.
The real “issue” with {serendipity_hookPlugin hook="frontend_comment" data=$commentform_entry} is that it's a “catchall hook”. It emits a lot of stuff which I sometimes would like to be able to emit in various places of the comment form. For instance, it would make sense to emit all the info boxes before the comment form, but it also emits actual form elements (like the captcha or the hidden field for spam bee, if used) which should be within the form. Some of these (like the hidden field for spam bee) can also be emitted using Smarty variables, but that needs to be configured properly in the related plugin by the end user.
YL
The real “issue” with {serendipity_hookPlugin hook="frontend_comment" data=$commentform_entry} is that it's a “catchall hook”. It emits a lot of stuff which I sometimes would like to be able to emit in various places of the comment form. For instance, it would make sense to emit all the info boxes before the comment form, but it also emits actual form elements (like the captcha or the hidden field for spam bee, if used) which should be within the form. Some of these (like the hidden field for spam bee) can also be emitted using Smarty variables, but that needs to be configured properly in the related plugin by the end user.
YL
Re: 2k11: commentform hook
I see, that is an argument to think about. I was focussed on the emoticons drop box only...
How about setting it to top and {capture} the output, to seperate different "plogs"...
How about setting it to top and {capture} the output, to seperate different "plogs"...
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Re: 2k11: commentform hook
I have no idea what that's supposed to mean.Timbalu wrote:How about setting it to top and {capture} the output, to seperate different "plogs"...
YL
Re: 2k11: commentform hook
It emits a lot of stuff which I sometimes would like to be able to emit in various places of the comment form.
likeHow about setting it to top and {capture} the output, to seperate different "plogs"...
Code: Select all
{capture name="splithook"}{the_hook...}{/capture} ... {if $splithook == 'something'}do it{/if}
Edit:
I once did this (without splitting) to append some output into sections tags without having them appear when empty.
Code: Select all
{capture name="pheh" assign="pluginHook_entries"}{serendipity_hookPlugin hook="entries_header" addData="$entry_id"}{/capture}
{if !empty($pluginHook_entries)}
<section id="section_hookPlugin_entries">
{$pluginHook_entries}
</section><!-- // "id:#section_hookPlugin_entries" end -->
{/if}
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Re: 2k11: commentform hook
I might be incredibly dumb here, but that still doesn't give me the ability to split up the output of the hook, does it?
YL
YL
Re: 2k11: commentform hook
Well, it might, ...if you know what usually comes by... but I have to declare this is a very dirty workaround (idea)!
Surely better would be, if you really like to split this, to append some unique "appended by xyzPlugin" o.e. var into the eventData Stream. Which then could easily could get sortet out by the template.
That effords Plugin changes, so we would need to know, what exactly is the feature request.
Surely better would be, if you really like to split this, to append some unique "appended by xyzPlugin" o.e. var into the eventData Stream. Which then could easily could get sortet out by the template.
That effords Plugin changes, so we would need to know, what exactly is the feature request.
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Re: 2k11: commentform hook
Then it's completely out of the question for a proposed new default template.Timbalu wrote:this is a very dirty workaround
I don't have a feature request. You're the one who wanted to change this in the first place.Timbalu wrote:That effords Plugin changes, so we would need to know, what exactly is the feature request.
YL
Re: 2k11: commentform hook
What is thisyellowled wrote:I don't have a feature request.
then?yellowled wrote:It emits a lot of stuff which I sometimes would like to be able to emit in various places of the comment form.
Regards,
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Ian
Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian