Topic: Quick jump drop list

This is really minor. When posting a new topic/reply, the 1st option is selected in the quick jump drop list. If possible, select the current forum instead.

Re: Quick jump drop list

Can't be done, since we cache the quickjump HTML.

Re: Quick jump drop list

But cache_quickjump_*.php files allow the selected option to be changed at include-time. I just tried this by adding "$forum_id = $fid;" before "require PUN_ROOT.'footer.php';" in post.php, and it seemed to work fine. Any reason why that wouldn't work?

Re: Quick jump drop list

That's what I get for not bothering to look at the code tongue
Added

5 (edited by qubertman 2008-03-18 02:15)

Re: Quick jump drop list

Rev: 1563 - Getting the following for viewforum.php and viewtopic.php:

Notice: Undefined variable: fid in C:\xampp\htdocs\punbb13\cache\cache_quickjump_2.php on line 5

Re: Quick jump drop list

Should be fixed

Re: Quick jump drop list

To make 'Post reply' behave the same way, the following was added to post.php:

$forum_id = $cur_posting['id'];

Is this safe to add?

Thanks.

Re: Quick jump drop list

Actually, that's a much better idea then what I was trying to do. tongue

Re: Quick jump drop list

Changed

Re: Quick jump drop list

Isn't that pretty much what I suggested in the first place? tongue

Adam Atlas wrote:

by adding "$forum_id = $fid;" before "require PUN_ROOT.'footer.php';" in post.php

Re: Quick jump drop list

Yeah. I guess I misread it. tongue

Re: Quick jump drop list

Please consider doing the same to edit.php, and delete.php. Thanks.

Re: Quick jump drop list

Done

Re: Quick jump drop list

edit.php - Missing a semicolon to the latest change.

Re: Quick jump drop list

Coding with a headache is annoying tongue
Fixed

Re: Quick jump drop list

5 changesets just to do one thing?! tongue