Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Best answer #33

Open
ceyhansuyu opened this issue Aug 27, 2015 · 7 comments
Open

Best answer #33

ceyhansuyu opened this issue Aug 27, 2015 · 7 comments

Comments

@ceyhansuyu
Copy link

How can I do best answer and topic solved?

topicsolved

@tierra
Copy link
Owner

tierra commented Aug 27, 2015

For it's initial release, it's not very flexible in regards to customization yet. I would recommend building your own extension that adds custom CSS in order to modify the style of the indicators. I haven't done this for my own indicators, but I have created an extension for my own board to manage custom CSS styles that you can use as an example:
https://github.com/tierra/wxforum-styles

@ceyhansuyu
Copy link
Author

Thank you.

@alekks
Copy link

alekks commented Oct 9, 2015

I'm not sure, but my interpretation of ceyhansuyu's request isn't about custom styling/formatting but to add a feature to mark a post within a thread as "Best answer". A bit like a lot of other forums work, e.g. Apple's support forums, where you can mark posts as "This helped me" etc.

@tierra
Copy link
Owner

tierra commented Oct 9, 2015

Well, if it's not about the style, it's about the text, and the text can already be customized (from the ACP) to say exactly what's in the screenshots. Maybe you're thinking of some third thing altogether though: something like upvotes?

@Galixte
Copy link
Contributor

Galixte commented Oct 9, 2015

Yes that which is particular that’s the text in the subject of the post is: “Best answer” while the text in the subject of the topic stays: “Solved”.

They want two texts, one for the topic title and one for the title of the best answer.

@tierra
Copy link
Owner

tierra commented Oct 9, 2015

Oh, right, I do remember thinking that originally, and I think that's why I chose to recommend custom CSS.

@tripflex
Copy link

tripflex commented Aug 8, 2016

For me this was very easy to do by modifying the code .. IMO it should be as easy as just adding another setting for "Selected Answer Phrase" and then in the function viewtopic_modify_post_row_subject in event/main_listener.php just use that instead of the default solved text.

For anybody else looking to do this, just go to this line in this file:
https://github.com/tierra/topicsolved/blob/master/event/main_listener.php#L294

And add:

$topic_data['forum_solve_text'] = 'Selected Answer';

or whatever you want it to say.

I'm a full time WordPress PHP dev but first time using PHPBB so i'm not sure if the fact that me using a custom theme makes a difference or not, or if doing this will change it anywhere else (from my tests it only did on the view topic page), but this should get you pointed in the right direction at least.

ALSO: I don't think this will work if you're using images, so keep that in mind

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants