docs icon indicating copy to clipboard operation
docs copied to clipboard

Change position property from top to right

Open molivier1988 opened this issue 1 year ago β€’ 2 comments
trafficstars

Description

Docs / css / concepts / position / terms / right.md

Assigned to add additional media to accurately represent code blocks. Noticed that code examples used top instead of `right.

Issue Solved

Issue #4074 changed position property from top to right

Type of Change

  • Editing an existing entry (fixing a typo, bug, issues, etc)

Checklist

  • [x] All writings are my own.
  • [x] My entry follows the Codecademy Docs style guide.
  • [x] My changes generate no new warnings.
  • [x] I have performed a self-review of my own writing and code.
  • [x] I have checked my entry and corrected any misspellings.
  • [x] I have made corresponding changes to the documentation if needed.
  • [x] I have confirmed my changes are not being pushed from my forked main branch.
  • [ x I have confirmed that I'm pushing from a new branch named after the changes I'm making.
  • [x] I have linked any issues that are relevant to this PR in the Issues Solved section.

molivier1988 avatar Jun 27 '24 16:06 molivier1988

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jun 27 '24 16:06 CLAassistant

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @molivier1988, could you please sign the CLA? Also, this entry requires an output image, could you please add that as well? You can store the image under the "media" folder in docs. Thank you! πŸ˜ƒ

mamtawardhani avatar Jun 28 '24 12:06 mamtawardhani

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @molivier1988, could you please sign the CLA? Also, this entry requires an output image, could you please add that as well? You can store the image under the "media" folder in docs. Thank you! πŸ˜ƒ

Hey @molivier1988 could you please make the required change at your earliest convenience?

mamtawardhani avatar Jul 08 '24 11:07 mamtawardhani

Hey @molivier1988 if you don't make the required changes within two days, unfortunately I'll have to close this PR and unassign you.

mamtawardhani avatar Jul 12 '24 12:07 mamtawardhani

Hi Mamta,

I’m still very new to all this, when you say make changes are you referring to the changes I recommended in my pull request?

KR

Mark

On 12 Jul 2024, at 13:09, Mamta Wardhani @.***> wrote:

Hey @molivier1988 https://github.com/molivier1988 if you don't make the required changes within two days, unfortunately I'll have to close this PR and unassign you.

β€” Reply to this email directly, view it on GitHub https://github.com/Codecademy/docs/pull/4848#issuecomment-2225447980, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV5P2FJRVFRGQM2YTLHTJHDZL7BRFAVCNFSM6AAAAABKAJ6QH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGQ2DOOJYGA. You are receiving this because you were mentioned.

molivier1988 avatar Jul 12 '24 17:07 molivier1988

Hi Mamta, I’m still very new to all this, when you say make changes are you referring to the changes I recommended in my pull request? KR Mark … On 12 Jul 2024, at 13:09, Mamta Wardhani @.***> wrote: Hey @molivier1988 https://github.com/molivier1988 if you don't make the required changes within two days, unfortunately I'll have to close this PR and unassign you. β€” Reply to this email directly, view it on GitHub <#4848 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV5P2FJRVFRGQM2YTLHTJHDZL7BRFAVCNFSM6AAAAABKAJ6QH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGQ2DOOJYGA. You are receiving this because you were mentioned.

Hello @molivier1988, yes, so basically this PR needs an output image as well as mentioned in the issue. So all you need to do is, include an image (while you are still on this same branch css-position-right and commit the changes and push them. Hope this helps 😁

mamtawardhani avatar Jul 12 '24 17:07 mamtawardhani

Hey @molivier1988, could you please try to push the changes in the next two days since it has been 3 weeks to this PR.

mamtawardhani avatar Jul 17 '24 08:07 mamtawardhani

Will work on it tonight

On 17 Jul 2024, at 09:18, Mamta Wardhani @.***> wrote:

Hey @molivier1988 https://github.com/molivier1988, could you please try to push the changes in the next two days since it has been 3 weeks to this PR.

β€” Reply to this email directly, view it on GitHub https://github.com/Codecademy/docs/pull/4848#issuecomment-2232713418, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV5P2FILLFLEFDXJM5LLUQDZMYSDXAVCNFSM6AAAAABKAJ6QH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSG4YTGNBRHA. You are receiving this because you were mentioned.

molivier1988 avatar Jul 17 '24 15:07 molivier1988

Will work on it tonight … On 17 Jul 2024, at 09:18, Mamta Wardhani @.***> wrote: Hey @molivier1988 https://github.com/molivier1988, could you please try to push the changes in the next two days since it has been 3 weeks to this PR. β€” Reply to this email directly, view it on GitHub <#4848 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV5P2FILLFLEFDXJM5LLUQDZMYSDXAVCNFSM6AAAAABKAJ6QH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSG4YTGNBRHA. You are receiving this because you were mentioned.

Sure thing! Let me know once you are done πŸ˜ƒ

mamtawardhani avatar Jul 18 '24 08:07 mamtawardhani

PR submitted, sorry for delaySent from my iPhoneOn 18 Jul 2024, at 09:58, Mamta Wardhani @.***> wrote:ο»Ώ

Will work on it tonight … On 17 Jul 2024, at 09:18, Mamta Wardhani @.***> wrote: Hey @molivier1988 https://github.com/molivier1988, could you please try to push the changes in the next two days since it has been 3 weeks to this PR. β€” Reply to this email directly, view it on GitHub <#4848 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV5P2FILLFLEFDXJM5LLUQDZMYSDXAVCNFSM6AAAAABKAJ6QH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSG4YTGNBRHA. You are receiving this because you were mentioned.

Sure thing! Let me know once you are done πŸ˜ƒ

β€”Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

molivier1988 avatar Jul 18 '24 09:07 molivier1988

PR submitted, sorry for delaySent from my iPhoneOn 18 Jul 2024, at 09:58, Mamta Wardhani @.> wrote:ο»Ώ Will work on it tonight … On 17 Jul 2024, at 09:18, Mamta Wardhani @.> wrote: Hey @molivier1988 https://github.com/molivier1988, could you please try to push the changes in the next two days since it has been 3 weeks to this PR. β€” Reply to this email directly, view it on GitHub <#4848 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV5P2FILLFLEFDXJM5LLUQDZMYSDXAVCNFSM6AAAAABKAJ6QH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSG4YTGNBRHA. You are receiving this because you were mentioned. Sure thing! Let me know once you are done πŸ˜ƒ β€”Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

No worries @molivier1988. Closing this PR since a new PR has been raised.

mamtawardhani avatar Jul 19 '24 07:07 mamtawardhani

Is there anything else I need to do?

molivier1988 avatar Aug 13 '24 18:08 molivier1988