Skip to content

added thread subject to post reports#9

Open
tzdlr wants to merge 4 commits into
SvePu:masterfrom
tzdlr:master
Open

added thread subject to post reports#9
tzdlr wants to merge 4 commits into
SvePu:masterfrom
tzdlr:master

Conversation

@tzdlr

@tzdlr tzdlr commented Feb 15, 2024

Copy link
Copy Markdown

just a small enhancement to include the thread subject, cuz on big boards you run out of overview when you just see repoted post by you will never find the corresponding report again :D

@Sama34

Sama34 commented Feb 15, 2024

Copy link
Copy Markdown

I think you did add the post subject in the sprintf() method, but is unused in the language string.

@tzdlr

tzdlr commented Feb 15, 2024

Copy link
Copy Markdown
Author

right, i just moved the post title at pos4, but thought it might be TOO much information in a thread, maybe creating a setting for it would solve the dilemma which one someone would prefer to use

@Sama34

Sama34 commented Feb 16, 2024

Copy link
Copy Markdown

Since people report posts and not threads, I would say that the post subject is more reliable (say, people actually change the post's subject).

Also, it seems this will generate a warning if $thread or $post aren't set. You need to move the full line into each switch case, as well as use a different language string for each case.

@tzdlr

tzdlr commented Feb 16, 2024

Copy link
Copy Markdown
Author

ouh yeah i missed that point, our board is strictly moderated so we dont have any, nor had any profile reports but therefore more thread reports regarding to board subject but you're right will revamp to suit all use cases!

… nicer overview with more information in thread title
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants