-
-
Notifications
You must be signed in to change notification settings - Fork 37.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: replace imgur to s3 for various files without conflict #36057
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -10,15 +10,15 @@ localeTitle: خلق نسيج عن طريق إضافة نمط دقيق كخلفي | |||||
<section id="description"> إحدى الطرق لإضافة الملمس والاهتمام إلى الخلفية وجعلها تبرز أكثر هي إضافة نمط دقيق. المفتاح هو التوازن ، لأنك لا تريد أن تبرز الخلفية أكثر من اللازم ، وأن تأخذها بعيدًا عن المقدمة. تدعم خاصية <code>background</code> وظيفة <code>url()</code> للربط بصورة للنسيج أو النمط المختار. يتم التفاف عنوان الارتباط بين علامات اقتباس داخل الأقواس. </section> | ||||||
|
||||||
## Instructions | ||||||
<section id="instructions"> باستخدام رابط من <code>https://i.imgur.com/MJAkxbh.png</code> ، تعيين <code>background</code> من صفحة كاملة مع <code>body</code> محدد. </section> | ||||||
<section id="instructions"> باستخدام رابط من <code>https://cdn-media-1.freecodecamp.org/imgr/MJAkxbh.png</code> ، تعيين <code>background</code> من صفحة كاملة مع <code>body</code> محدد. </section> | ||||||
|
||||||
## Tests | ||||||
<section id='tests'> | ||||||
|
||||||
```yml | ||||||
tests: | ||||||
- text: يجب أن يكون لعنصر <code>body</code> الخاص بك خاصية <code>background</code> معيّنة إلى <code>url()</code> بالرابط المحدد. | ||||||
testString: 'assert(code.match(/background:\s*?url\(\s*("|"|)https:\/\/i\.imgur\.com\/MJAkxbh\.png\1\s*\)/gi), "Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.");' | ||||||
testString: 'assert(code.match(/background:\s*?url\(\s*("|"|)https:\/\/cdn-media-1\.freecodecamp\.org\/imgr\/MJAkxbh\.png\1\s*\)/gi), "Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.");' | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
``` | ||||||
|
||||||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -12,7 +12,7 @@ One way to add texture and interest to a background and have it stand out more i | |||||
|
||||||
## Instructions | ||||||
<section id='instructions'> | ||||||
Using the url of <code>https://i.imgur.com/MJAkxbh.png</code>, set the <code>background</code> of the whole page with the <code>body</code> selector. | ||||||
Using the url of <code>https://cdn-media-1.freecodecamp.org/imgr/MJAkxbh.png</code>, set the <code>background</code> of the whole page with the <code>body</code> selector. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
</section> | ||||||
|
||||||
## Tests | ||||||
|
@@ -21,7 +21,7 @@ Using the url of <code>https://i.imgur.com/MJAkxbh.png</code>, set the <code>bac | |||||
```yml | ||||||
tests: | ||||||
- text: Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link. | ||||||
testString: assert(code.match(/background:\s*?url\(\s*("|'|)https:\/\/i\.imgur\.com\/MJAkxbh\.png\1\s*\)/gi), 'Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.'); | ||||||
testString: assert(code.match(/background:\s*?url\(\s*("|'|)https:\/\/cdn-media-1\.freecodecamp\.org\/imgr\/MJAkxbh\.png\1\s*\)/gi), 'Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.'); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
``` | ||||||
|
||||||
|
@@ -52,7 +52,7 @@ tests: | |||||
```html | ||||||
<style> | ||||||
body { | ||||||
background: url("https://i.imgur.com/MJAkxbh.png"); | ||||||
background: url("https://cdn-media-1.freecodecamp.org/imgr/MJAkxbh.png"); | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need to change this, I believe we can just enter any URL there, the current one does not even make sense as a BG. We'll need to update all languages' files There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it's fine as is. The URL does have to be an image for the camper to see a change. Also, this image does create the subtle change that the instructions talk about. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @thecodingaviator, thanks for the catch. If it is alright with you, I would rather differ content modifications specially in the curriculum to another PR since it would require a further discussion. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since we're doing it to the Egnlish file, we'll need to do it to the other languages too. If you want to do it in this PR, fine enough, I'm totally fine if you want another file There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you @thecodingaviator, I took note of suggestions, and I will address them in a separate PR. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If everything else other than changing the background look acceptable, please feel free to merge this PR. |
||||||
} | ||||||
</style> | ||||||
``` | ||||||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -10,15 +10,15 @@ localeTitle: Criar textura adicionando um padrão sutil como imagem de plano de | |||||
<section id="description"> Uma maneira de adicionar textura e interesse a um plano de fundo e destacá-lo é adicionar um padrão sutil. A chave é o equilíbrio, já que você não quer que o fundo se destaque muito e tire o primeiro plano. A propriedade <code>background</code> suporta a função <code>url()</code> para vincular a uma imagem da textura ou padrão escolhido. O endereço do link é colocado entre aspas entre parênteses. </section> | ||||||
|
||||||
## Instructions | ||||||
<section id="instructions"> Usando o url de <code>https://i.imgur.com/MJAkxbh.png</code> , defina o <code>background</code> de toda a página com o seletor de <code>body</code> . </section> | ||||||
<section id="instructions"> Usando o url de <code>https://cdn-media-1.freecodecamp.org/imgr/MJAkxbh.png</code> , defina o <code>background</code> de toda a página com o seletor de <code>body</code> . </section> | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
## Tests | ||||||
<section id='tests'> | ||||||
|
||||||
```yml | ||||||
tests: | ||||||
- text: Seu elemento <code>body</code> deve ter uma propriedade <code>background</code> definida como <code>url()</code> com o link fornecido. | ||||||
testString: 'assert(code.match(/background:\s*?url\(\s*("|"|)https:\/\/i\.imgur\.com\/MJAkxbh\.png\1\s*\)/gi), "Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.");' | ||||||
testString: 'assert(code.match(/background:\s*?url\(\s*("|"|)https:\/\/cdn-media-1\.freecodecamp\.org\/imgr\/MJAkxbh\.png\1\s*\)/gi), "Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.");' | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
``` | ||||||
|
||||||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -10,15 +10,15 @@ localeTitle: Crear textura agregando un patrón sutil como imagen de fondo | |||||
<section id="description"> Una forma de agregar textura e interés a un fondo y hacer que se destaque más es agregar un patrón sutil. La clave es el equilibrio, ya que no desea que el fondo se destaque demasiado y quita el primer plano. La propiedad de <code>background</code> compatible con la función <code>url()</code> para enlazar a una imagen de la textura o patrón elegido. La dirección del enlace está entre comillas dentro de los paréntesis. </section> | ||||||
|
||||||
## Instructions | ||||||
<section id="instructions"> Usando la url de <code>https://i.imgur.com/MJAkxbh.png</code> , establezca el <code>background</code> de toda la página con el selector de <code>body</code> . </section> | ||||||
<section id="instructions"> Usando la url de <code>https://cdn-media-1.freecodecamp.org/imgr/MJAkxbh.png</code> , establezca el <code>background</code> de toda la página con el selector de <code>body</code> . </section> | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
## Tests | ||||||
<section id='tests'> | ||||||
|
||||||
```yml | ||||||
tests: | ||||||
- text: El elemento del <code>body</code> debe tener una propiedad de <code>background</code> establecida en una <code>url()</code> con el enlace dado. | ||||||
testString: 'assert(code.match(/background:\s*?url\(\s*("|"|)https:\/\/i\.imgur\.com\/MJAkxbh\.png\1\s*\)/gi), "Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.");' | ||||||
testString: 'assert(code.match(/background:\s*?url\(\s*("|"|)https:\/\/cdn-media-1\.freecodecamp\.org\/imgr\/MJAkxbh\.png\1\s*\)/gi), "Your <code>body</code> element should have a <code>background</code> property set to a <code>url()</code> with the given link.");' | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
``` | ||||||
|
||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.