Skip to content

added yt-summaries page#132

Merged
Arker123 merged 7 commits intoArker123:mainfrom
EdgarThorpe:2021csb1169/sof-86-youtube-summaries-page-for-showing-examples
Apr 15, 2024
Merged

added yt-summaries page#132
Arker123 merged 7 commits intoArker123:mainfrom
EdgarThorpe:2021csb1169/sof-86-youtube-summaries-page-for-showing-examples

Conversation

@EdgarThorpe
Copy link
Collaborator

@EdgarThorpe EdgarThorpe commented Mar 12, 2024

-Added YTSummaries Page (jsx file and css file)

@linear
Copy link

linear bot commented Mar 12, 2024

Copy link
Owner

@Arker123 Arker123 left a comment

Choose a reason for hiding this comment

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

Could you please eliminate redundant comments and include a description for the PR? Thank you.

@Arker123 Arker123 enabled auto-merge April 10, 2024 08:28
@Arker123 Arker123 added the UI label Apr 11, 2024
Copy link
Owner

@Arker123 Arker123 left a comment

Choose a reason for hiding this comment

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

Almost good to go. Thanks

Comment on lines +138 to +142
/* .Category span {
display: block;
margin-bottom: 20px;
color: #555;
} */ No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
/* .Category span {
display: block;
margin-bottom: 20px;
color: #555;
} */

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Deleted this portion of code, as it wasn't needed.

@Arker123
Copy link
Owner

Also could you please include a description for this PR. Thanks

Copy link
Collaborator

@Dhruv-Singh-Negi Dhruv-Singh-Negi left a comment

Choose a reason for hiding this comment

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

lgtm

@Arker123 Arker123 merged commit 9e99b9d into Arker123:main Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants