bugSavane - Bugs: bug #7780, spaces at the beggining of lines...

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

bug #7780: spaces at the beggining of lines in the mail messages

Submitted by:  Paul Pogonyshev <doublep>
Submitted on:  Sun 19 Nov 2006 07:33:53 PM UTC  
 
Category: Web Frontend: TrackersStatus: Need Info
Severity: 1 - WishPriority: A - Later
Assigned to: NoneOpen/Closed: Open
Release: SVNPlanned Release: 
Reproducibility: NonePrivacy: Public

(Jump to the original submission Jump to the original submission)

Mon 20 Nov 2006 10:01:23 PM UTC, comment #11:

I think it is wordwrap() that is used.

I think it is likely that wordwrap() was actually designed to behave the way it does. If I had to write it, would probably do the same: assume we can trash one white space when putting a line break, but not more than one, because if there are more, you can assume it is on purpose. Because typography rules in English states that you have only one white space between the dot and the first letter of the next sentence (like it is in most european languages, if not all).

Mathieu Roy <yeupou>
Project Administrator
Mon 20 Nov 2006 09:56:25 PM UTC, comment #10:

In comments to wordwrap(): "I found that wordwrap deletes the spaces it wraps on."

So, either you use another function or this feature was added in a later PHP version. In the former case, I suggest to use wordwrap() instead.

Paul Pogonyshev <doublep>
Project Member
Mon 20 Nov 2006 09:49:20 PM UTC, comment #9:

Uh, that indeed changes things. Maybe it should be considered a bug in the (is it wordwrap?) underlying function then.

Paul Pogonyshev <doublep>
Project Member
Mon 20 Nov 2006 09:32:32 PM UTC, comment #8:

Yes, but that's a PHP function. So the bug report should be made against PHP I think, shouldn't it?

Mathieu Roy <yeupou>
Project Administrator
Mon 20 Nov 2006 09:18:59 PM UTC, comment #7:

Yes, I said it is my habit to put two spaces between sentences. Ideally, wrap() would replace all consecutive spaces with a line break, not just the first space.

Paul Pogonyshev <doublep>
Project Member
Mon 20 Nov 2006 08:46:30 PM UTC, comment #6:

The database content is exactly

The two spaces where in the original content. The line break was likely added by wrap(). I guess that's a bug in wrap() :) I don't know.

Mathieu Roy <yeupou>
Project Administrator
Mon 20 Nov 2006 08:40:33 PM UTC, comment #5:

There is a leading space in this line of the comment:

I agree this is of very low priority.

Paul Pogonyshev <doublep>
Project Member
Mon 20 Nov 2006 08:34:10 PM UTC, comment #4:

I must say I still dont get it :P

Notifications does not include space at the beginning of lines.
There are white space only when listing the fields, for the purpose of having values aligned.

Mathieu Roy <yeupou>
Project Administrator
Mon 20 Nov 2006 08:19:10 PM UTC, comment #3:

Hum, that's weird, I cannot open your file. I filled a bug (bug #7796). In the meantime, can you reupload your file after renaming it?

Mathieu Roy <yeupou>
Project Administrator
Mon 20 Nov 2006 08:10:19 PM UTC, comment #2:

I attach a mail I received after my post to a tracker.

(file #1538)

Paul Pogonyshev <doublep>
Project Member
Mon 20 Nov 2006 08:50:12 AM UTC, comment #1:

Hum, I'm not sure to understand. Could you elaborate? :)

Mathieu Roy <yeupou>
Project Administrator
Sun 19 Nov 2006 07:33:53 PM UTC, original submission:

I have a habit to separate sentences with two space (Emacs convension.) It would be nice, it mail notifications from trackers didn't include a space (I guess it's the second one) at the beggining of a line.

Paul Pogonyshev <doublep>
Project Member

 

Attached Files
file #1539:  example-mail-with-a-leading-space added by doublep (3kB - application/octet-stream)
file #1538:  [bug #7779] Going back in time with GNU Go added by doublep (3kB - application/octet-stream - Example mail)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by yeupou (Posted a comment)
  • -unavailable- added by doublep (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 20 Nov 2006 08:28:16 PM UTCdoublepAttached File-=>Added example-mail-with-a-leading-space, #1539
    Mon 20 Nov 2006 08:10:19 PM UTCdoublepAttached File-=>Added [bug #7779] Going back in time with GNU Go, #1538
    Mon 20 Nov 2006 08:50:12 AM UTCyeupouCategoryBackend=>Web Frontend: Trackers
      StatusConfirmed=>Need Info
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup