Wedge

Public area => Bug reports => The Pub => Archived fixes => Topic started by: oOo--STAR--oOo on April 21st, 2012, 10:24 PM

Title: Thoughts not display @ on initial post
Post by: oOo--STAR--oOo on April 21st, 2012, 10:24 PM
Hey,

I just replied to nao's statement about baidu bots, and when I initially submitted it, it didn't display @nao
When I refreshed it then displayed the @nao, but didn't display it when I initially posted it.

I am using google chrome.
Title: Re: Thoughts not display @ on initial post
Post by: Arantor on April 21st, 2012, 10:25 PM
That's not a bug, that's exactly as designed.
Title: Re: Thoughts not display @ on initial post
Post by: oOo--STAR--oOo on April 21st, 2012, 10:27 PM
Quote from Arantor on April 21st, 2012, 10:25 PM
That's not a bug, that's exactly as designed.
Alright cool, if that's how it was as designed.
Thought it was a bug lol.
Title: Re: Thoughts not display @ on initial post
Post by: Arantor on April 21st, 2012, 10:28 PM
Hence the 'refresh to interact' too ;)
Title: Re: Thoughts not display @ on initial post
Post by: Nao on April 21st, 2012, 10:59 PM
I can probably make it show everything as needed though. This hack was a quick fix when I developed the feature. I just don't remember why I failed to make it work even though it sends the thought back via Ajax...
Title: Re: Thoughts not display @ on initial post
Post by: oOo--STAR--oOo on April 21st, 2012, 11:03 PM
Quote from Nao on April 21st, 2012, 10:59 PM
I can probably make it show everything as needed though. This hack was a quick fix when I developed the feature. I just don't remember why I failed to make it work even though it sends the thought back via Ajax...
Do you prependTo to the HTML of the response ?
The response is probably different than the display all. As in you not included the @ in the response.
Title: Re: Thoughts not display @ on initial post
Post by: Nao on April 21st, 2012, 11:45 PM
Let's be clear. The thought system is not 100% finished. It still has a few bugs (try to post a new thought from the sidebar, then try to 'Modify' it immediately without reloading the page... It won't show you the current thought, even though there's one.)
I just don't think it's a huge priority right now. Priorities should basically be whatever will be hard to modify *after* we go live...

For instance, the work I'm doing now on the language files... It's something that's going to be annoying to translators if it's done *after* the first public release.
Title: Re: Thoughts not display @ on initial post
Post by: live627 on May 31st, 2012, 09:33 PM
Can this be moved?
Title: Re: Thoughts not display @ on initial post
Post by: Arantor on May 31st, 2012, 09:37 PM
It's not quite perfect yet.

The dynamically-added item doesn't have the link image back to the profile area, and for some reason the dynamically added name does not have the user link colouring, e.g. if I reply to a thought, the @ name in the reply does not take on the colouring for consultants or admins.
Title: Re: Thoughts not display @ on initial post
Post by: Nao on May 31st, 2012, 11:59 PM
Yup, I noticed that...
Actually, I miss the days when the @ names were all shown in gray... It made the overall look a bit less confusing. (But less logical as well.)

But in the end, all I can say is that it shouldn't be behaving that way. AFAIK the returned Ajax thought goes through output buffers so it should be transforming any member links to use their group color...
Title: Re: Thoughts not display @ on initial post
Post by: Nao on June 13th, 2012, 02:47 PM
That's a fixed issue, btw...