Skip to content
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

[#12693] Excess padding on edit course details component #12737

Merged
merged 5 commits into from
Feb 10, 2024

Conversation

abdullahsohailcs
Copy link
Contributor

Fix #12693

Outline of Solution

Change was made to the following file having relative URL as under:
src\web\app\components\course-edit-form\course-edit-form.component.html

Following steps were taken to solve this bug/issue.

  1. Read the documentation on how to contribute,
  2. Set up the project on my laptop.
  3. Ran the project in the browser and navigated to the page where the bug was present.
  4. Then moved to the code base in Visual Studio Code and found the page and the component where bug was present.
  5. Understood the bug and fixed it.

PS: This is my first open-source contribution. Before this contribution, I have never worked on this stack. I am looking forward to making more contributions. Thank you all 😀!

Image of updated UI is as under.
Capture

Copy link
Contributor

@weiquu weiquu left a comment

Choose a reason for hiding this comment

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

Do take a look at the instructor-course-edit-page component as mentioned in the comment

Copy link
Contributor

@weiquu weiquu left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for contributing

@weiquu weiquu added the s.FinalReview The PR is ready for final review label Feb 10, 2024
@domlimm domlimm self-requested a review February 10, 2024 03:46
Copy link
Contributor

@domlimm domlimm left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for your contribution!

@domlimm domlimm merged commit 2bd2367 into TEAMMATES:master Feb 10, 2024
9 checks passed
cedricongjh pushed a commit to cedricongjh/teammates that referenced this pull request Feb 20, 2024
…AMMATES#12737)

* Update course-edit-form.component.html

Removed unnecessary padding

* Update course-edit-form.component.spec.ts.snap

also added the change in snap file

* Revert "Update course-edit-form.component.spec.ts.snap"

This reverts commit 897ac22.

* Revert "Update course-edit-form.component.html"

This reverts commit d74ac3c.

* Update instructor-course-edit-page.component.html

---------

Co-authored-by: Abdullah-Sohail100 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s.FinalReview The PR is ready for final review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Excess padding on edit course details component
3 participants