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

feat : Space between links #9590

Closed
wants to merge 5 commits into from
Closed

feat : Space between links #9590

wants to merge 5 commits into from

Conversation

Vijaykv5
Copy link
Member

Fixes Issue #9406

Changes proposed

On links page, Added a margin left on the heading of links.
Resized the Edit button to make it better look, Also changed the Alignment of the edit button by adding Respective padding between the divs.

Before:
Screenshot 2023-10-23 at 7 17 00 PM

After:
Screenshot 2023-10-23 at 7 09 59 PM

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Copy link
Member

@sital002 sital002 left a comment

Choose a reason for hiding this comment

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

The .env.docker shouldn't be changed please revert those changes.

@Vijaykv5
Copy link
Member Author

@sital002 I have Changed it. Please check

@Vijaykv5 Vijaykv5 requested a review from sital002 October 23, 2023 17:06
@sital002
Copy link
Member

@sital002 I have Changed it. Please check

There are no changes on the files

@Vijaykv5
Copy link
Member Author

is it fine now @sital002 or I need to make any more changes?

@Vijaykv5
Copy link
Member Author

I think now it's fixed @sital002

Copy link
Member

Choose a reason for hiding this comment

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

There is only one file .env.docker which shouldn't be changed please revert these changes.

@Vijaykv5 Vijaykv5 closed this by deleting the head repository Oct 23, 2023
@Vijaykv5
Copy link
Member Author

I have made a new PR in order to clear any confusions
check out #9592

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.

2 participants