322808 – Spell checking doesn't work anymore

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 322808 - Spell checking doesn't work anymore

Summary: Spell checking doesn't work anymore

Status:	CLOSED FIXED

Alias:	None

Product:	z_Archived
Classification:	Eclipse Foundation
Component:	PDT (show other bugs)
Version:	unspecified
Hardware:	PC Windows XP

Importance:	P2 normal with 1 vote (vote)
Target Milestone:	---
Assignee:	PHP UI
QA Contact:

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-08-16 11:44 EDT by Silver Zachara
Modified:	2020-05-14 11:08 EDT (History)
CC List:	4 users (show)

See Also:

Attachments
fix for broken spellchecking around php code (1.07 KB, patch) 2012-10-25 07:54 EDT, Iljas Khayrullin	jacek.pospychala: iplog+	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Silver Zachara

2010-08-16 11:44:27 EDT

Hi,

spell checking doesn't work anymore in PDT editor.

I have defined default "English" spellchecking and my own dictionary too.

In all other editors spellchecking works fine.

Comment 1 Silver Zachara

2010-10-01 05:51:22 EDT

Hi,

any news about this issue.

I lack this feature in my PHP files very very much.

To Roy:
pls, can change priority of this ticket to P1?

I think, that in the first maintaince realease of SR1 should be fixed this ticket too:
http://dev.eclipse.org/mhonarc/lists/pdt-dev/msg01511.html

Thank you

Comment 2 Iljas Khayrullin

2012-10-25 07:54:18 EDT

Created attachment 222827 [details]
fix for broken spellchecking around php code

Spellcheck works well inside PHP strings and comments in PHP files.
Bus spellcheck doesn't work in html content in PHP files.
The patch fixes this.

Comment 3 Jacek Pospychala

2012-10-25 10:06:06 EDT

hi Iljas,
thank you for the patch!

Comment 4 Jacek Pospychala

2012-10-25 12:13:57 EDT

Iljas thanks again for the patch!

I simplified it a bit and committed to Git repo.
Please give it a try and let me know if you see any issues.

Comment 5 Iljas Khayrullin

2012-11-22 01:20:14 EST

Thank you, Jacek!
It works well.

Comment 6 Sylvia Tancheva

2015-06-09 08:38:30 EDT

Closing