Skip to content

Made UI of extension#160

Merged
n1ti merged 3 commits intomainfrom
2021csb1169/sof-108-create-ui-for-the-chrome-extension
Mar 26, 2024
Merged

Made UI of extension#160
n1ti merged 3 commits intomainfrom
2021csb1169/sof-108-create-ui-for-the-chrome-extension

Conversation

@EdgarThorpe
Copy link
Collaborator

@EdgarThorpe EdgarThorpe commented Mar 22, 2024

  • Made the UI for the extension.

@linear
Copy link

linear bot commented Mar 22, 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.

Please provide a description to this PR

Copy link
Collaborator Author

@EdgarThorpe EdgarThorpe left a comment

Choose a reason for hiding this comment

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

  • Changed the directory structure according to the requested changes.
  • Added the mac specific cache files in gitignore.
  • UI completed and relative path of the files updated.

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.

Please provide a description to this PR

Copy link
Owner

Choose a reason for hiding this comment

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

This file is still present, can you remove this please

Copy link
Owner

Choose a reason for hiding this comment

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

Do we really need a copy of the logo?

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.

lgtm, thanks

@n1ti n1ti merged commit c2fc725 into main Mar 26, 2024
@Arker123 Arker123 deleted the 2021csb1169/sof-108-create-ui-for-the-chrome-extension branch May 28, 2024 09:26
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.

3 participants