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

feat: custom-notes feature in webui #80

Closed
wants to merge 22 commits into from
Closed

Conversation

MasterK0927
Copy link
Collaborator

@MasterK0927 MasterK0927 commented Sep 11, 2024

Allows users to jot down and save personal notes directly within the browser. The feature includes two main implementations: an in-toolbar button and a dedicated WebUI page.

1. Notes Button in the Toolbar:

  • Button in the browser toolbar renders a popup when clicked.

  • Popup allows users to enter and save custom notes or text.

  • Notes are stored persistently across sessions using the browser's local storage.

2. Dedicated WebUI Page:

  • Custom Notes feature has been extended to a dedicated WebUI page.

  • Inkludes option for Summarising and Rephrasing the Notes.

API Calls are tested in two ways :

  • Uses native JavaScript APIs to perform API calls directly. Certain checks restricting WebUI API calls are disabled to enable this.

  • Backend communicates with WebUI page via Mojo IPC mechanism, delegating calls to the JavaScript frontend. API calls are routed through the browser’s C++ backend using GURL.

Additonal Comments

  • Tested on Ubuntu 24.1
  • Synced with 130

}, []);

const loadNotes = () => {
customNotesAPI.pageHandler.getAllNotes().then((response: { notes: Note[] }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

is it possible to use async await here?

setNotesList([...notesList, note]);
setNote('');
// Add note to the backend via mojom handler
customNotesAPI.pageHandler.addNote(note).then(() => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here. using .then might introduce complications during development

@MasterK0927
Copy link
Collaborator Author

MasterK0927 commented Dec 30, 2024

  • Changing branch name to more meaningful name
  • Branch is in sync with 130
  • Adding more meaningful description and heading.

@MasterK0927 MasterK0927 changed the title added new notes feature with ui in c++ feat: custom-notes feature in webui Dec 30, 2024
@MasterK0927 MasterK0927 deleted the sign-final-keshav branch December 30, 2024 10:41
@MasterK0927
Copy link
Collaborator Author

Migrated to #89

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.

2 participants