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 #12736

Closed
wants to merge 2 commits into from
Closed

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

wants to merge 2 commits into from

Conversation

abdullahsohailcs
Copy link
Contributor

@abdullahsohailcs abdullahsohailcs commented Feb 9, 2024

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

github-actions bot commented Feb 9, 2024

Hi @abdullahsohailcs, thank you for your interest in contributing to TEAMMATES!
However, your PR does not appear to follow our contribution guidelines:

  • Title must start with the issue number the PR is fixing in square brackets, e.g. [#<issue-number>]

Please address the above before we proceed to review your PR.

@abdullahsohailcs abdullahsohailcs changed the title [Issue #12693] Excess padding on edit course details component [#12693] Excess padding on edit course details component Feb 9, 2024
@abdullahsohailcs abdullahsohailcs deleted the padding-removed branch February 9, 2024 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Excess padding on edit course details component
1 participant