Invision Community 4: SEO, prepare for v5 and dormant account notifications Matt November 11, 2024Nov 11
Posted November 11, 201212 yr 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&section=post&do=edit_post&f={$topic['forum_id']}&t={$topic['tid']}&p={$post['post']['pid']}&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 alloweReference 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.
November 11, 201212 yr I oppose the idea , moving edit button on a different place within thread will cause bunch of problems regarding UI consistency .
November 11, 201212 yr 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.
November 12, 201212 yr 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.
November 12, 201212 yr 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.
September 24, 201311 yr 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.
Archived
This topic is now archived and is closed to further replies.