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

Call paste handler #35

Merged
merged 7 commits into from
Sep 18, 2024
Merged

Call paste handler #35

merged 7 commits into from
Sep 18, 2024

Conversation

psrpinto
Copy link
Member

@psrpinto psrpinto commented Sep 12, 2024

This PR makes it so that we call the pasteHandler() when a user clicks on an element. We don't do anything with the data yet, that will be implemented in future PRs.

Build tooling changes

Previously, webpack was always being run in production mode, which meant that in development the code was minified and there were no source maps. Now, webpack can also be called in development mode. We now have the following scripts:

  • build:production: builds both firefox and chrome versions in production mode, into build/production
  • build:firefox: builds firefox version only, in development mode, into build/firefox
  • build:chrome: builds chrome version only, in development mode, into build/chrome

The development builds now have source maps, which means we can debug using the browser's debugger.

Screen capture

Screenshot 2024-09-17 at 15 14 15

@psrpinto psrpinto changed the base branch from trunk to highlight-content September 12, 2024 12:16
@psrpinto psrpinto self-assigned this Sep 12, 2024
@psrpinto
Copy link
Member Author

@akirk @ashfame Even though this is still draft, I welcome your early feedback.

@akirk
Copy link
Member

akirk commented Sep 12, 2024

Looks very good! We want to use blocks mode indeed.

@ashfame
Copy link
Member

ashfame commented Sep 17, 2024

Pretty good indeed!

Base automatically changed from highlight-content to trunk September 17, 2024 10:48
@psrpinto psrpinto marked this pull request as ready for review September 17, 2024 14:21
@psrpinto psrpinto requested review from akirk and ashfame and removed request for akirk and ashfame September 17, 2024 14:21
Copy link
Member

@ashfame ashfame left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Member

@akirk akirk left a comment

Choose a reason for hiding this comment

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

Nice! Well done!

src/parser/cleanHtml.ts Show resolved Hide resolved
@psrpinto psrpinto merged commit e045947 into trunk Sep 18, 2024
3 checks passed
@psrpinto psrpinto deleted the paste-handler branch September 18, 2024 16:35
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