Skip to content

Conversation

@MasterK0927
Copy link

@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
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
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