Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Ace fixes #420

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Ace fixes #420

wants to merge 4 commits into from

Conversation

adamjimenez
Copy link
Contributor

Fix issue where pressing undo after initial load, clears the editor.
Single undo for text that was typed over. Before it would delete and restore in two undos.
Upped initial undo limit from 50 to 500.

@coveralls
Copy link

coveralls commented Mar 25, 2023

Pull Request Test Coverage Report for Build 4518587577

Details

  • 6 of 6 (100.0%) changed or added relevant lines in 4 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.03%) to 65.598%

Files with Coverage Reduction New Missed Lines %
lib/text-operation.js 1 93.12%
Totals Coverage Status
Change from base Build 835215972: 0.03%
Covered Lines: 2314
Relevant Lines: 3381

💛 - Coveralls

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants