Re: [Mantisbt-dev] Some patches for your consideration

[Jeroen L. <jl...@ca...>]

On Wed, 3 Mar 2004 01:38:08 -0500, Raymond DeRoo <rd...@de...> wrote:

> Greetings! Some of  you may know as [RainMkr] from Freenode (#php,
> #mysql and now #mantishelp). I stumbled across the project about a week
> ago when I needed something that was lightweight, and not nearly as
> ugly as bugzilla. :) As I was trying to get the code working (yea I
> know, RTFM) I came up with a list of questions "Why was this done that
> way?" and "Was this the intended effect?" So I thought I would start
> delving into the code. The PHP code is nicely formatted, but when
> needing to look at the html source... That left a little to be desired.
> So I modified my copy of the code to meet my needs/desires.
>
> In the future I promise to keep any patches smaller, but I just got
> rolling on things. At any rate, these patches need to applied to the
> 0.18.2 tree with root.patch in ./mantis-0.18.2 and core.patch in
> ./mantis-0.18.2/core
>
> Now I realize that this may seem pointless, and does require more
> bandwidth to send each page. But I think the benefit will be seen while
> debugging some strange layout issue that can creep up.

Looks like a good change, although I'd call the function 
string_insert_display_tabs. One question though: why did you move the 
html_title call?

Jeroen