Landscape descriptions contain "\n"

Bug #678417 reported by Matthew Gates
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Stellarium
Fix Released
Low
Unassigned

Bug Description

\n should be removed / turned into a space in the landscape descriptions. Better to do it in the code, as there are many landscapes already out there which include this.

Related branches

Changed in stellarium:
importance: Undecided → Low
milestone: none → 0.10.6
Revision history for this message
Bogdan Marinov (daggerstab) wrote :

The 0.9 series used it as a new line character. The description uses HTML in the 0.10 series, so it should be replaced with <br/>.

Revision history for this message
Matthew Gates (matthew-porpoisehead) wrote : Re: [Bug 678417] Re: Landscape descriptions contain "\n"

In the 0.9.x series, the \n was used as an end if line character
because there was no auto-wrap in the description text field in the
old GUI (so primitive!).
<br /> looks like a paragraph delimiter, if we substitute \n with <br
/> we're going to end up with some pretty weird looking descriptions.

For example:

description = The Sirene Astronomical Observatory\nLagarde d'Apt (84),
France\n\nPanoramic view of installations. Previously the site\nwas
used as a nuclear lauching pad. Now days,\nSirene accepts everybody
for astronomical\nobservations.\n\nPhotos : March 29, 2008\nLandscape
creation : May 11, 2008\nLast update : May 18,
2008\n\nwww.obs-sirene.com

The \n is not being used as a paragraph delimiter.

It might be worthwhile replacing "\n\n" with "<br />" before the "\n"
with "" replacement. What do you think?

Matthew

On 22 November 2010 11:53, Bogdan Marinov <email address hidden> wrote:
> The 0.9 series used it as a new line character. The description uses
> HTML in the 0.10 series, so it should be replaced with <br/>.
>
> --
> Landscape descriptions contain "\n"
> https://bugs.launchpad.net/bugs/678417
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Changed in stellarium:
status: New → Fix Committed
Changed in stellarium:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.