Skip to content

feat(demo): allow to edit class hash - #583

Merged
m-kus merged 1 commit into
03-04-feat_demo_add_subsystem_health_indicators_to_explorerfrom
02-26-feat_demo_allow_to_edit_class_hash
Mar 4, 2026
Merged

m-kus merged 1 commit into
03-04-feat_demo_add_subsystem_health_indicators_to_explorerfrom
02-26-feat_demo_allow_to_edit_class_hash

Conversation

@m-kus

@m-kus m-kus commented Feb 26, 2026 •

Copy link
Copy Markdown

Add editable pool class hash functionality

This change enables users to modify the pool class hash directly in the UI through a double-click to edit interface. The class hash can now be edited inline with keyboard shortcuts (Enter to apply, Escape to cancel) or via an Apply button.

Changes:

  • Added state management for class hash editing with editingClassHash, classHashDraft, and classHash state variables
  • Implemented inline editing UI that appears when double-clicking the class hash display
  • Added CSS styling for the edit interface including input field and button styling
  • Updated pool selector and deploy functionality to use the dynamic class hash instead of the static config value
  • Enhanced code element styling with cursor and word-break properties for better UX

This change is Reviewable

@m-kus
m-kus marked this pull request as ready for review February 26, 2026 16:49
@m-kus
m-kus requested a review from noa-starkware February 26, 2026 16:50
@github-actions

github-actions Bot commented Feb 26, 2026 •

Copy link
Copy Markdown

Deploy preview for starknet-privacy-demo ready!

✅ Preview
https://starknet-privacy-demo-kuzhv8xji-keep-starknet-strange.vercel.app

Built with commit 3697236.
This pull request is being automatically deployed with vercel-action

@noa-starkware noa-starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

:lgtm:

@noa-starkware made 1 comment.
Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on m-kus).

@m-kus
m-kus changed the base branch from 02-26-feat_demo_transaction_builder to graphite-base/583 March 4, 2026 13:23
@m-kus
m-kus force-pushed the graphite-base/583 branch from 35ff33c to f5e8d24 Compare March 4, 2026 13:57
@m-kus
m-kus force-pushed the 02-26-feat_demo_allow_to_edit_class_hash branch 2 times, most recently from 18c29ed to b70abd5 Compare March 4, 2026 14:15
@m-kus
m-kus force-pushed the 02-26-feat_demo_allow_to_edit_class_hash branch from b70abd5 to 3697236 Compare March 4, 2026 14:49
@m-kus
m-kus changed the base branch from graphite-base/583 to 03-04-feat_demo_add_subsystem_health_indicators_to_explorer March 4, 2026 14:49
@m-kus
m-kus merged commit 04c793b into 03-04-feat_demo_add_subsystem_health_indicators_to_explorer Mar 4, 2026
2 of 3 checks passed
@m-kus
m-kus deleted the 02-26-feat_demo_allow_to_edit_class_hash branch March 4, 2026 19:29
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.

2 participants