Last modified: 2014-04-07 13:56:06 UTC
Created attachment 6991 [details] Wiki_r61227_2010-01-21_NTOC.pdf Reporting against Babaco Release : r61227 Steps to Reproduce :: 1) Select a random page 2) Add headings <<Headings are not immediately showing in the Navigable TOC>> 3) Click on a heading << Nothing changes>> 4) Click on Hide Content and again click on Show content 5) Click on a heading <<shows the heading, sometimes highlighting the previous heading>> Please refer to the attachment for a detail description Expected Outcome:: Header items should display in Navigable TOC as soon as it is added to the editor Test Environment:: Browser (User-Agent): Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.7) Gecko/20091221 Firefox/3.5.7 (.NET CLR 3.5.30729) Browser (User-Agent): Opera/9.63 (Windows NT 6.0; U; en) Presto/2.1.1
How long was the lag approximately in Calcey's environment?
(In reply to comment #1) > How long was the lag approximately in Calcey's environment? For clarity: there's a 1/4 second delay in that the TOC is only refreshed 1/4 second after you stop typing or do certain other things. This report seems to suggest that the delay is substantially longer, though.
Another thought: When you add a heading via the toolbar, it isn't added to the TOC. It gets picked up as soon as you start to edit the default text that is placed into the text, but when the text is added as a result of the click event on the toolbar's heading drop down, a change event doesn't seem to get fired. Is this the desirable behavior? Or should we be adding it to the TOC as soon as it's in the text area, regardless of wether it's the default text or not?
(In reply to comment #3) > Another thought: When you add a heading via the toolbar, it isn't added to the > TOC. It gets picked up as soon as you start to edit the default text that is > placed into the text, but when the text is added as a result of the click event > on the toolbar's heading drop down, a change event doesn't seem to get fired. > > Is this the desirable behavior? Or should we be adding it to the TOC as soon as > it's in the text area, regardless of wether it's the default text or not? This used to work, probably lacking a .trigger( 'change' ) in encapsulateSelection. Will poke at it.
Fixed in r61402.
(In reply to comment #5) > Fixed in r61402. Oops, this rev only fixes the bug Adam mentioned in comment #3, not the bug as filed.
We are now listening for encapsulateSelection as well when looking for headings, which resolves this quite nicely.
Created attachment 7028 [details] Wiki_r61537_2010-01-27_Headings.pdf
(In reply to comment #8) > Created an attachment (id=7028) [details] > Wiki_r61537_2010-01-27_Headings.pdf Verified against r61537 Different browsers display differently on creating and highlighting header items. Please refer the attachment. Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.7) Gecko/20091221 Firefox/3.5.7 (.NET CLR 3.5.30729) Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.0; SLCC1; .NET CLR 2.0.50727; Media Center PC 5.0; .NET CLR 3.5.21022; .NET CLR 3.5.30729; .NET CLR 3.0.30618) Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US) AppleWebKit/532.0 (KHTML, like Gecko) Chrome/3.0.195.38 Safari/532.0 Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US) AppleWebKit/525.28 (KHTML, like Gecko) Version/3.2.2 Safari/525.28.1 Opera/9.63 (Windows NT 6.0; U; en) Presto/2.1.1
This seems to be a mixture of different issues, most of which were fixed. Could you retry after I update the software on prototype?
Solved this in r61584 by adding <div> to <br> conversion in getContents.
Created attachment 7038 [details] Wiki_r61577_2010-01-28_Headings.pdf
Comment on attachment 7038 [details] Wiki_r61577_2010-01-28_Headings.pdf deleting the attachment
Created attachment 7055 [details] Wiki_r61584_2010-02-01_Headings.pdf
Verified against the latest version hosted on http://test.wikipedia.org NTOC is updated as user enters heading from toolbar. NTOC is highlighted when a heading is selected.
Created attachment 7083 [details] Wiki_prod_2010-02-08_headers.pdf Test link : http://en.wikipedia.org/wiki/Main_Page Steps: 1. Edit a page 2. Add a heading 3. Move to the end of the added heading and press enter 4. Add another heading Browsers : Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US) AppleWebKit/532.0 (KHTML, like Gecko) Chrome/3.0.195.38 Safari/532.0 Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US) AppleWebKit/525.28 (KHTML, like Gecko) Version/3.2.2 Safari/525.28.1 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/530.17 (KHTML, like Gecko) Version/4.0 Safari/530.17
(In reply to comment #16) > Created an attachment (id=7083) [details] > Wiki_prod_2010-02-08_headers.pdf > > Test link : http://en.wikipedia.org/wiki/Main_Page > > Steps: > 1. Edit a page > 2. Add a heading > 3. Move to the end of the added heading and press enter > 4. Add another heading > > Browsers : > > Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US) AppleWebKit/532.0 (KHTML, like > Gecko) Chrome/3.0.195.38 Safari/532.0 > > Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US) AppleWebKit/525.28 (KHTML, like > Gecko) Version/3.2.2 Safari/525.28.1 > > Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US) AppleWebKit/530.17 (KHTML, like > Gecko) Version/4.0 Safari/530.17 Therefore reopening the bug
Tested link: http://prototype.wikimedia.org/deployment-en/ Version : r62529 Bug verification depend on Bug 22543
(In reply to comment #18) > Tested link: http://prototype.wikimedia.org/deployment-en/ > Version : r62529 > > Bug verification depend on Bug 22543 ... which I marked as FIXED 20 minutes before you posted this comment.
Unassigning default assignments. http://article.gmane.org/gmane.science.linguistics.wikipedia.technical/54734
Closing old tickets about Navigable TOC which is not worked on anymore.