Hi,

Just for the record: Stephane and I went through the code on IRC and
found that the rendering code was in settings/js/log.js in the
addEntries() method.

Cheers,

Vincent

On 07/05/2014 10:53 PM, Stéphane Vergeylen wrote:
>  
>
> Hi Vincent, 
>
> Thank you to welcome me on the mailing list yesterdy. I just began to
> try solving my first junior job by picking bug #9147. 
>
> May I request some guidance for resolving this ? 
>
> I posted this in the comments of #9147 by metionning your pseudo : 
>
> My analyse of the bug is the following : 
>
>  - The last cell of the table containing the timestamp looks fine
> because there is a class="date" which adds "white-space: nowrap;" to the
> cell 
>
>  - When playing with the <select>, the ajax loads a new table back where
> this class="date" is not present, which cause the line break. 
>
> Solution : 
>
>  - Add the class="date" in the processing of the table in the ajax
> request (settings/js/admin.js) 
>
>  - But, there, at line 49, I can't find the corresponding HTML markup. I
> find only OC.Log.reload(). Where is the markup ? 
>
> Thank you for guidance... 
>
> And if I want to report this (tiny) change, what should I do ? Commit in
> a new branch based on stable7 and then require a pull ? 
>
>
> _______________________________________________
> Devel mailing list
> [email protected]
> http://mailman.owncloud.org/mailman/listinfo/devel

_______________________________________________
Devel mailing list
[email protected]
http://mailman.owncloud.org/mailman/listinfo/devel

Reply via email to