Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.

Move mention of download in progress at h2 level #1208

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DelazJ
Copy link
Contributor

@DelazJ DelazJ commented Nov 28, 2023

Currently, there is a big "Thank you". Let's complete with information that the download is ongoing...

Copy link
Contributor Author

@DelazJ DelazJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While getting in touch with Kontur guys, any issues with merging this? Or let's just hold back till their feedback?

@@ -96,9 +96,8 @@
}
else if (currentPage.match(/site\/forusers\/(download|alldownloads)\.html/g)){
$($('body')[0]).append('<div id="thankyou"> <div id="thankyoucontent">'+
'<h2>{{ _("Thank you!") }}</h2> <p/>'+
'<h2>{{ _("Thank you! Your freshly baked copy of QGIS is downloading.") }}</h2> <p/>'+
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'<h2>{{ _("Thank you! Your freshly baked copy of QGIS is downloading.") }}</h2> <p/>'+
'<h2>{{ _("Thank you! Your freshly baked copy of QGIS is downloading...") }}</h2> <p/>'+

'<div class="container"><div class="span8">'+
'<p><b>{{ _("Your freshly baked copy of QGIS is downloading.") }}</b></p>'+
'<p>{{ _("QGIS is free of charge, and will always be free of charge if downloaded from QGIS.org.") }}</p>'+
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'<p>{{ _("QGIS is free of charge, and will always be free of charge if downloaded from QGIS.org.") }}</p>'+
'<p>{{ _("Due to the file size the download may take some time. Please be patient.") }}</p>'+
'<p>{{ _("QGIS is free of charge, and will always be free of charge if downloaded from QGIS.org.") }}</p>'+

Copy link
Contributor

@strk strk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good to me. Let's see if my review helps moving this further (I also have pending PRs :)

@rduivenvoorde
Copy link
Contributor

rduivenvoorde commented Jan 25, 2024

One of the drawbacks of doing this very small edits on such important pages, is that all translations break.

I've discussed this with Harrissou in a remote session (and told him that I'm not in favour of pulling this, because this message is important).

But others can (or if @DelazJ really wants this, I will do (or he can do himself)).

PS: another reason not to edit such important pages (anymore) is that QGIS is busy with a new website(design/content).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants