Jump to content

(option for) Edit Button at Top

Featured Replies

Posted

This is going back in Feedback since no one had a glitch-free solution on how to code this.

The best idea was this:

ACP - Look and Feel - Your skin - Topic View - post
Find

<p class='posted_info desc lighter ipsType_small'>

Add above

				<if test="$post['post']['_can_edit'] === TRUE">
					<a href='{parse url="module=post&amp;section=post&amp;do=edit_post&amp;f={$topic['forum_id']}&amp;t={$topic['tid']}&amp;p={$post['post']['pid']}&amp;st={$this->request['st']}" base="publicWithApp"}' title='{$this->lang->words['post_edit_title']}' class='ipsButton_secondary edit_post right' id='edit_post_{$post['post']['pid']}'>{$this->lang->words['post_edit']}</a>
				</if>




However, this allowe

Reference for tracker: 3.4 Beta4 truncated my post here ^


Retyped:
However, this allowed the edit button at the bottom to be used only once. After that, the edit button at the top is the only that can be used unless the page is refreshed.

Also, I made a poll for this and a few people were against it, so if this was made an option in the ACP then there'd be no reason to oppose this addition.

I oppose the idea , moving edit button on a different place within thread will cause bunch of problems regarding UI consistency .

Umm what are you trying to do? Just move the edit button to a different location in the post bit? There is nothing hard about that.

  • Author

Umm what are you trying to do?


Well I am trying to get a working version of what I already have, which is an edit button at top including the default location at the bottom. However, the one at the bottom only works for the first edit because of some glitch in the code I was using.

They use the same HTML id's, which must be unique within a document. Simply duplicating that link is not going to work correctly due to javascript issues, as you are seeing.

As a general rule, we do not add settings which specify where to place links. We could literally have billions of settings, confusing everyone (novices and advanced users alike) and still not have the settings people specifically want. Moving items around within the interface is what the skin system is designed for.

  • 10 months later...
  • Author

Bumping this because it still works for 3.4.5 and (I believe) is a helpful addition to posts.

Thanks to newbie LAC for coming up with this.

Disclaimer: It is occasionally buggy in that when doing repeated edits on the same post that the smaller edit button on the bottom will not work. But that bug is barely detectable and is nothing refresh can't fix. Regardless, the button on top always works.

editbutton_zpsdf114f66.jpg

Archived

This topic is now archived and is closed to further replies.

Recently Browsing 0

  • No registered users viewing this page.