Skip to content

Asset editor improvements#224

Open
fredsa wants to merge 27 commits intosehugg:masterfrom
fredsa:asseteditor
Open

Asset editor improvements#224
fredsa wants to merge 27 commits intosehugg:masterfrom
fredsa:asseteditor

Conversation

@fredsa
Copy link
Contributor

@fredsa fredsa commented Mar 1, 2026

  • Reformat affected files prior to other commits
  • Fix asset undo (Asset Editor doesn't undo properly #213)
  • Implement asset editor redo
  • Create distinct undo for each asset edit
  • Limit asset editor to undos in current session
  • Fix stacking "no more undo/redo" alerts
  • Ignore ';;' that is a new ';;{…};;' asset block
  • Asset editor displays line number ranges and header
  • Click line number range to jump to highlighted lines
  • Show errors for assets with incorrect number of values
  • Fix bad edits when source has different length values
  • Fix length new data values written
  • Align thumbnails to the top to prevent visual jumps
  • Dismiss highlighted lines on user interaction
  • Replace affected asset range, not entire file contents
  • Asset undo inside source editor shows affected lines
  • Likely fixes [Possible bug] Asset editor gets reset #62

@fredsa fredsa changed the title Asset editor fix undo #213, add redo Asset editor improvements Mar 2, 2026
@fredsa
Copy link
Contributor Author

fredsa commented Mar 2, 2026

Updated PR title and the initial comment

@fredsa
Copy link
Contributor Author

fredsa commented Mar 2, 2026

Rebased commits on origin/master

@fredsa
Copy link
Contributor Author

fredsa commented Mar 2, 2026

Last two commits add:

  • affordance for asset line numbers in asset editor
  • clickable asset editor decorations in source code

@fredsa fredsa force-pushed the asseteditor branch 2 times, most recently from 7635b5d to d44745a Compare March 4, 2026 22:58
@fredsa
Copy link
Contributor Author

fredsa commented Mar 4, 2026

Rebased on origin/master

@fredsa fredsa force-pushed the asseteditor branch 2 times, most recently from 2a8a45b to 514820b Compare March 6, 2026 02:42
@fredsa
Copy link
Contributor Author

fredsa commented Mar 6, 2026

  • Fix validateAssetData, add max reachable offset
    calculation, fixes NES issues with chr_generic.s
  • Add validateAssetData support for rletag
  • Fix broken asset editing and undo when asset source
    has not been previously opened manually, for example
    when reloading the #asseteditor page directly
  • Fix palette color undo/redo

@fredsa
Copy link
Contributor Author

fredsa commented Mar 6, 2026

  • Implement #asseteditor/filename/lineno support,
    and highlight selected asset on navigation

@fredsa
Copy link
Contributor Author

fredsa commented Mar 6, 2026

  • place the "asset editor" decoration after header instead at the end of the line

fredsa added 14 commits March 6, 2026 17:38
- For each detected asset, display its line number
  and header snippet for reference.
- When asset data cannot be found or the number
  of values detected is incorrect, and inline
  error message is displayed instead of a (broken)
  asset editor.
align-items: flex-start
Use bpw or verilog spec to pad correct number of chars.
Do not treat a new asset block
as the end of a previous block
fredsa added 13 commits March 6, 2026 17:38
Asset editor replaces text ranges instead of
the entire source file. Undo inside the source
code now scrolls asset editor edits into view,
and selects the text being edited, so that what
is being (un)edited is much clearer. Also, undo
history is no longer burdened by full file edits.
Add calculation of maximum reachable offset.
Fixes errors with `chr_generic.s`
Fix broken asset edit and undo when the asset
source file has not been opened manually, for
example when reloading the #asseteditor page
directly.
- Track selected asset in hash
- Highlight selected asset on navigation
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.

[Possible bug] Asset editor gets reset

1 participant