Several bugs

Started by cichlasoma, May 07, 2010, 01:05:35 PM

Previous topic - Next topic

cichlasoma

Hi, I post several bug reports, I'm well aware that the ones at the end of the list are not very serious... :-)

1. When I try to uncommnent two comment blocks in html at once with Format/Uncomment selection, the closing tag of the first comment block ("—>") remains in the text.

2. When I open Project/Properties and click OK, HE crashes with the same error messages each time, I've filled and sent an automated bug report.

3. When I set default document encoding to 1250 (Central european (Windows)), create a html file, save it, close and open again, it displays in the encoding 1252 (Western european (Windows)) and some special characters are dispalyed incorrectly of course. It does not happen with other formats I tried (I mean other than html) – they display correctly in 1250 after having been reopened.

4. On my Asus EEEPC 1000H with screen resolution 1024 × 600, word wrapping of HipoEDIT config files works not properly when "By window edge" mode is set – it breaks lines although much greater parts of them (or even whole these lines) could be displayed on the respective row on the screen. See http://i39.tinypic.com/2z9llk1.png ("By window edge" mode was set in Syntax settings/HipoEDIT config/Font and colors).

5. When word wrapping is on, the tripple click selects just the current row, not the whole line/paragraph. (Is it a bug or a feature? Alt + Down selects the whole line.)

6. I use czech keyboard and when I write digits using Shift + "the key which would write the digit on an english keyboard", i get messages like "The key combination (Shift + ě) is not a command" in the status bar (digits are typed correctly nevertheless). (I get no such messages when writing capital letters.)

7. The help file comming with HE 1.48 contains an outdated screenshot of the Options menu in the section on Code highlighting. I was not able to find where to set the default font referred to by "Use from Default" option in the Syntax settings/Font and colors of individual syntaxes for some time and the helpfile was not much helpful (see http://i40.tinypic.com/dgkwsk.png ). (I've found where to set the default font already...)

alex

#1
Hi cichlassoma,

the main problem here, that you put all bug reports together in once post ;) I wrote about this already to another user, in such way it is very difficult to track solving, answering detailed, and then to search for other users. At then end such post became huge , multi paged mixed discussion...

Quote1. When I try to uncommnent two comment blocks in html at once with Format/Uncomment selection, the closing tag of the first comment block ("—>") remains in the text.
Such way of uncommenting for multiple commented areas was not designed, because it is mostly impossible to recognize comment symbols, if you have selected commented area not exactly or not completely (imaging that your selection stops somewhere inside of the commented area). Is should work if you select it exactly. So, this is more feature request, than bug report. But I will check, if it possible to solve in easy way.

Quote2. When I open Project/Properties and click OK, HE crashes with the same error messages each time, I've filled and sent an automated bug report.
It is a new bug introduced with 1.48. I got your crash report, thanks, and can reproduce it by myself. Probably because of this I need to update 1.48 installer once more instead of placing this into 1.49.

Quote3. When I set default document encoding to 1250 (Central european (Windows)), create a html file, save it, close and open again, it displays in the encoding 1252 (Western european (Windows)) and some special characters are dispalyed incorrectly of course. It does not happen with other formats I tried (I mean other than html) – they display correctly in 1250 after having been reopened.
Can you please send me the example file. Does it have somewhere inside code page defined (encoding=" or charset )?
It can be problem with encoding auto detection (search on forum for encoding) or something else.

Quote4. On my Asus EEEPC 1000H with screen resolution 1024 × 600, word wrapping of HipoEDIT config files works not properly when "By window edge" mode is set – it breaks lines although much greater parts of them (or even whole these lines) could be displayed on the respective row on the screen. See http://i39.tinypic.com/2z9llk1.png ("By window edge" mode was set in Syntax settings/HipoEDIT config/Font and colors).
From screenshot I see it seems to me, that wrap is done by page width. Which I by default 75. Check that this is not like this (display page width indicator) also please check that you have not changed document specific wrap (menu View->Editor).
And one more thing, if used font is not mono-spaced, wrap position calculated not accurate, this is known "limitation". But I do not think that it is a reason in this case.

Quote5. When word wrapping is on, the tripple click selects just the current row, not the whole line/paragraph. (Is it a bug or a feature? Alt + Down selects the whole line.)
This was designed like this. In word wrap mode line selection on triple is done by sub line. There was no specail thoughts about this, I just decided to do like this. If you have good arguments, this can be changed in 1.49.

Quote6. I use czech keyboard and when I write digits using Shift + "the key which would write the digit on an english keyboard", i get messages like "The key combination (Shift + ě) is not a command" in the status bar (digits are typed correctly nevertheless). (I get no such messages when writing capital letters.)
Hm. I thought this problem was already fixed... By such warning HE tries to inform you that such shortcut does not exist, if it thinks that you have pressed shortcut. Such check is not doe if only Shift pressed. Problem occurs only with such extended characters, because in this case, for some reason, Ctrl button is also marked as pressed (as far as I remember, or Alt). So HE thinks that this is a shortcut. I will check what is possible to get here, but do not know if will succeed, this is not a first try :)

Quote7. The help file comming with HE 1.48 contains an outdated screenshot of the Options menu in the section on Code highlighting. I was not able to find where to set the default font referred to by "Use from Default" option in the Syntax settings/Font and colors of individual syntaxes for some time and the helpfile was not much helpful (see http://i40.tinypic.com/dgkwsk.png ). (I've found where to set the default font already...)
Yes, this is true. Help has not been updated for a long time already. Usually recent information can be found on forum. I will try to update it with 1.50. But of course, as always, my problem here is time.
HippoEDIT team
[url="http://www.hippoedit.com/"]http://www.hippoedit.com/[/url]

cichlasoma

#2
Hi alex,
I'm sorry about combining several reports in one post, will do better next time. If you consider it appropriate, please answer each bug in a special thread.

Quote from: alex on May 07, 2010, 06:10:34 PM
Can you please send me the example file. Does it have somewhere inside code page defined (encoding=" or charset )?
It can be problem with encoding auto detection (search on forum for encoding) or something else.

It happens evens with a file containing just "ěščřžýáíé" when saved as xxx.html. It doesn't happen when saved xxx.txt, xxx.py and so on.

QuoteFrom screenshot I see it seems to me, that wrap is done by page width. Which I by default 75. Check that this is not like this (display page width indicator) also please check that you have not changed document specific wrap (menu View->Editor).
And one more thing, if used font is not mono-spaced, wrap position calculated not accurate, this is known "limitation". But I do not think that it is a reason in this case.

For some reason, on this machine, HE refuses to display the page width for "HipoEDIT config" type and it uses another font than courier which is set for it. (For xml, there are exactly the same settings as for HippoEDIT config and there are no document-specific settings.)
(I reinstalled HippoEDIT recently deleting all the settings from the previous installation.)

cichlasoma

#3
Quote from: cichlasoma on May 07, 2010, 07:31:19 PM
For some reason, on this machine, HE refuses to display the page width for "HipoEDIT config" type and it uses another font than courier which is set for it. (For xml, there are exactly the same settings as for HippoEDIT config and there are no document-specific settings.)
(I reinstalled HippoEDIT recently deleting all the settings from the previous installation.)

I add some screenshots to document it:

http://i44.tinypic.com/jh77t4.png
http://i43.tinypic.com/2945lpx.png
http://i42.tinypic.com/24670aq.png

alex

With word wrap position the reason is a font (and I take my previous words back :) ).
Even if Courier marked as mono-spaced, it is not mono-spaced really. Characters width with different styles (bold, italic) is different. because of this resulting wrapping position calculated wrong (HE uses simple algorithm by just multiplication of character count to average width of symbols with bold+italic font).
If you will switch to Courier New, it will work correctly. So this is a known bug, maybe I will have a time to come back to this topic after 1.50 release. As I wrote some where in forum before, for source code editor support of non mono-spaced fonts is not high priority issue (without mono-spaced font block selection will now work). But, sure, this will be fixed some when.

About encoding change:
- Are you sure, that encoding of new HTML document was 1250. Check in status bar. In my case even with default document encoding set to 1250, it was some 23689 (sorry forget exact code page already, it was set as default for html in html_spec.xml default="iso-8859-1", will be now utf-8, inherited from xml_spec).
- Another side of problem, default HTML template does not have charset define, so it is always auto detected heuristically, if encoding is not explicitly given by user (with menu or during save)
- So probably the reason is not enough data for heuristic algorithm.

I have removed default encoding from html spec. So now all HTML, which does not have charset information inside will be saved as UTF-8, and problem with wrong display of characters should be solved. To set preferred encoding, just provide charset tag or change encoding explicitly.
HippoEDIT team
[url="http://www.hippoedit.com/"]http://www.hippoedit.com/[/url]

cichlasoma

#5
Quote from: alex on May 07, 2010, 06:10:34 PM
This was designed like this. In word wrap mode line selection on triple is done by sub line. There was no specail thoughts about this, I just decided to do like this. If you have good arguments, this can be changed in 1.49.

(My argument would be, all the other editors implementing triple clicke I know select a paragraph on it and I am used to it at least :) .
Besides it, I find it logical that one needs to select a pargraph (which is an intentionally created section of the text) more often than a row (which is a random rusult of the current screen width).)

cichlasoma

Quote from: alex on May 08, 2010, 02:11:16 AM
With word wrap position the reason is a font (and I take my previous words back :) ).
Even if Courier marked as mono-spaced, it is not mono-spaced really. Characters width with different styles (bold, italic) is different. because of this resulting wrapping position calculated wrong (HE uses simple algorithm by just multiplication of character count to average width of symbols with bold+italic font).
If you will switch to Courier New, it will work correctly.

You are right. Thank you!

cichlasoma

QuoteAbout encoding change:
- Are you sure, that encoding of new HTML document was 1250. Check in status bar.

I'm sorry, in status bar, I see, the encoding is 1252. It creates new html files with that encoding although the default is set to 1250. Then it reopens them in 1252 again but some characters ("ě" at least) are displayed in a wrong way I don't know why. When I change the document's encoding to 1250 explicitly after it's created, it's reopened with 1250 and characters are displayed correctly.

cichlasoma

#8
Quote from: alex on May 08, 2010, 02:11:16 AM
With word wrap position the reason is a font (and I take my previous words back :) ).
Even if Courier marked as mono-spaced, it is not mono-spaced really.

BTW, is there any reason for HippoEDIT config files having set Courier font by default? (In other words, is it a "bug" or a feature?)
(I was not right saying that xml files had the same settings - they use default font by default (as do all the other formats) which is Courier New (and I hadn't changed the setting).)

alex

QuoteThis was designed like this. In word wrap mode line selection on triple is done by sub line. There was no specail thoughts about this, I just decided to do like this. If you have good arguments, this can be changed in 1.49.

(My argument would be, all the other editors implementing triple clicke I know select a paragraph on it and I am used to it at least :) .
Besides it, I find it logical that one needs to select a pargraph (which is an intentionally created section of the text) more often than a row (which is a random rusult of the current screen width).)
My concerns here were not about line verse paragraph, but line verse sub-line then wrapping is on.
I think selection of line on triple click is OK, because there is no paragraphs in source code editors, and single line have same weight in word-line-paragraph selection. And it is consistent. Because double click selects word.
So, here I will do no change.

QuoteBTW, is there any reason for HippoEDIT config files having set Courier font by default? (In other words, is it a "bug" or a feature?)
(I was not right saying that xml files had the same settings - they use default font by default (as do all the other formats) which is Courier New (and I hadn't changed the setting).)
No it is not a bug but also not a feature. Default font face for hippoedit config is Consolas (which is mono-spaced). Maybe you do not have it installed, and when have checked settings for hippoedit config, next font in the list was selected (which was probably Courier) and after you have pressed OK, or Apply because of some change, it was taken. If font is not selected, default will be Courier New. Consolas for config files was taken by my whim :) I found it installed on all XP machines and also mono-spaced. But maybe it is not installed on netbooks..
HippoEDIT team
[url="http://www.hippoedit.com/"]http://www.hippoedit.com/[/url]

cichlasoma

#10
Quote from: alex on May 09, 2010, 02:22:51 AMMy concerns here were not about line verse paragraph, but line verse sub-line then wrapping is on.
I think selection of line on triple click is OK, because there is no paragraphs in source code editors, and single line have same weight in word-line-paragraph selection. And it is consistent. Because double click selects word.
So, here I will do no change.
I'm sorry, but here, I don't understand you very well. By "paragraph", I meant a line (i.e. a part of a text closed with "\n"). If you have word wrapping on, it's displayed as a paragraph. (HE is source code editor in the first place for sure, but it's pretty suitable for editing plain text documents as well.)
Anyway - the concern is line/paragraph versus row/subline here. And I argue that the latter is a random result of the current screen width and one doesn't need to select it as often as the whole line which is a logical unit.

QuoteI think selection of line on triple click is OK, because there is no paragraphs in source code editors,
(But HE doesn't select lines, but sublines or "screen rows".)

Quoteand single line have same weight in word-line-paragraph selection. And it is consistent. Because double click selects word.
I'm sorry, I have no idea what you mean by this. Couldn't you rephrase it somehow, please?


QuoteNo it is not a bug but also not a feature. Default font face for hippoedit config is Consolas (which is mono-spaced). Maybe you do not have it installed, and when have checked settings for hippoedit config, next font in the list was selected (which was probably Courier) and after you have pressed OK, or Apply because of some change, it was taken. If font is not selected, default will be Courier New. Consolas for config files was taken by my whim :) I found it installed on all XP machines and also mono-spaced. But maybe it is not installed on netbooks..

You are right, I have not Consolas installed on my EEEPC by default. But Courier is selected for HippoEDIT config files on this computer right from the beginning, not first after I check the settings and press OK.

alex

OK if now we find out that discussion is a about logical line and wrapped screen line, I agree. I will change the logic to select on triple click logical line.

QuoteYou are right, I have not Consolas installed on my EEEPC by default. But Courier is selected for HippoEDIT config files on this computer right from the beginning, not first after I check the settings and press OK.
No, it is not. At least was not selected initially. You can check installed files and will see inside, that font is Consolas. Because font was not found, Windows has substituted it with some system font (probably FixedSys), which we can see in one of your first screenshots. Later, after you have opened settings for HippoEDIT config, HippoEDIT has selected next font in the font combo box (sorted by name) and this by chance was Courier, because Consolas was not found in the list. If you then press Cancel, no changes will be made, but if you will change something (even not a font name) you will get new font name saved as Courier.
HippoEDIT team
[url="http://www.hippoedit.com/"]http://www.hippoedit.com/[/url]

cichlasoma

#12
Quote from: alex on May 09, 2010, 04:51:59 PM
No, it is not. At least was not selected initially. You can check installed files and will see inside, that font is Consolas. Because font was not found, Windows has substituted it with some system font (probably FixedSys), which we can see in one of your first screenshots. Later, after you have opened settings for HippoEDIT config, HippoEDIT has selected next font in the font combo box (sorted by name) and this by chance was Courier, because Consolas was not found in the list. If you then press Cancel, no changes will be made, but if you will change something (even not a font name) you will get new font name saved as Courier.

I reinstalled HE deleting all old config files and without even opening Options, HippoEDIT config files were displayed with Courier and with broken word wrapping, right from the beginning. I believe you, Courier is not selected, but it's used for HippoEDIT config files by default when you have not Consolas installed, even before you open Options.

alex

This is Courier: http://i41.tinypic.com/5xsbw8.png
and what you have is http://i43.tinypic.com/2945lpx.png FixedSys.

I also displayed page width of 75 columns, and it is visible that is calculated right to the screen width.
HippoEDIT team
[url="http://www.hippoedit.com/"]http://www.hippoedit.com/[/url]

cichlasoma

#14
Quote from: alex on May 09, 2010, 11:59:45 PM
This is Courier: http://i41.tinypic.com/5xsbw8.png
and what you have is http://i43.tinypic.com/2945lpx.png FixedSys.

Well, maybe, what I have in HippoEDIT config files from the beginning after installation, is not Courier. But definitelly, I have another font in this file type than the default one (Courier New) and when I set Word Wrap to Window Edge for "All Files" later, it wraps Hippoedit config files in a wrong way.
But strangely, it seems, that what I have is courier which seems to be displayed in my system otherwise than i yours: http://i44.tinypic.com/148f5vm.png

QuoteI also displayed page width of 75 columns, and it is visible that is calculated right to the screen width.

And how do you see it? For me, screen width is not displayed in HippoEDIT config files at all when I set it to be displayed. (see http://i44.tinypic.com/jh77t4.png )