mirror of
https://github.com/wickedest/Mergely.git
synced 2026-02-26 11:07:56 +08:00
Compare commits
12 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
6ed235c4b0 | ||
|
|
27778f0840 | ||
|
|
3651df9ad6 | ||
|
|
f1e58f58fe | ||
|
|
5cb3946182 | ||
|
|
69ecf76fe6 | ||
|
|
93eee752a7 | ||
|
|
00a79170a8 | ||
|
|
c90edb3150 | ||
|
|
1a2ee6af3d | ||
|
|
aa496078fd | ||
|
|
d6d8d06acc |
42
CHANGELOG.md
42
CHANGELOG.md
@@ -1,3 +1,45 @@
|
||||
## [5.4.5](https://github.com/wickedest/Mergely/compare/v5.4.4...v5.4.5) (2026-02-23)
|
||||
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* **#222:** Fixed issue with poor rendering performance and marker misalignment ([27778f0](https://github.com/wickedest/Mergely/commit/27778f0840895bf5e029d8c962e99890b8a0a8f6)), closes [#222](https://github.com/wickedest/Mergely/issues/222) [#227](https://github.com/wickedest/Mergely/issues/227)
|
||||
|
||||
## [5.4.4](https://github.com/wickedest/Mergely/compare/v5.4.3...v5.4.4) (2026-02-23)
|
||||
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Updated examples README.md ([f1e58f5](https://github.com/wickedest/Mergely/commit/f1e58f58fea85628b1dd345b98b23923f05d8c2e))
|
||||
|
||||
## [5.4.3](https://github.com/wickedest/Mergely/compare/v5.4.2...v5.4.3) (2026-02-23)
|
||||
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Removed whitespace from example ([69ecf76](https://github.com/wickedest/Mergely/commit/69ecf76fe6066ded1efca06f9ee0ad985e8ed047))
|
||||
|
||||
## [5.4.2](https://github.com/wickedest/Mergely/compare/v5.4.1...v5.4.2) (2026-02-23)
|
||||
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* Removed whitespace from example ([00a7917](https://github.com/wickedest/Mergely/commit/00a79170a8d4b701d6da98c1d6ea921b592fb3d1))
|
||||
|
||||
## [5.4.1](https://github.com/wickedest/Mergely/compare/v5.4.0...v5.4.1) (2026-02-23)
|
||||
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
* **#225:** Fixes rendering issue when there are multiple editors ([1a2ee6a](https://github.com/wickedest/Mergely/commit/1a2ee6af3d7d87160e72e8faca5e38cf01b36fcb))
|
||||
|
||||
# [5.4.0](https://github.com/wickedest/Mergely/compare/v5.3.6...v5.4.0) (2025-11-15)
|
||||
|
||||
|
||||
### Features
|
||||
|
||||
* **#220:** Exposes Mergely's instance of CodeMirror as Mergely.CodeMirror ([#221](https://github.com/wickedest/Mergely/issues/221)) ([d6d8d06](https://github.com/wickedest/Mergely/commit/d6d8d06accaa4fa3a4352b95620e3e456226d444)), closes [#220](https://github.com/wickedest/Mergely/issues/220)
|
||||
|
||||
## [5.3.6](https://github.com/wickedest/Mergely/compare/v5.3.5...v5.3.6) (2024-10-11)
|
||||
|
||||
|
||||
|
||||
@@ -1,6 +1,5 @@
|
||||
# Mergely examples
|
||||
|
||||
|
||||
## Mergely development server
|
||||
|
||||
To run the hot dev-server for development, you need to run:
|
||||
|
||||
@@ -2,9 +2,6 @@ require('codemirror/addon/selection/mark-selection.js');
|
||||
require('codemirror/lib/codemirror.css');
|
||||
require('../src/mergely.css');
|
||||
|
||||
|
||||
|
||||
|
||||
document.onreadystatechange = function () {
|
||||
if (document.readyState !== 'complete') {
|
||||
return;
|
||||
|
||||
@@ -14,7 +14,6 @@ the quick brown fox
|
||||
jumped over the lazy dog
|
||||
`;
|
||||
|
||||
|
||||
document.onreadystatechange = function () {
|
||||
if (document.readyState !== 'complete') {
|
||||
return;
|
||||
|
||||
4
package-lock.json
generated
4
package-lock.json
generated
@@ -1,12 +1,12 @@
|
||||
{
|
||||
"name": "mergely",
|
||||
"version": "5.3.6",
|
||||
"version": "5.4.5",
|
||||
"lockfileVersion": 3,
|
||||
"requires": true,
|
||||
"packages": {
|
||||
"": {
|
||||
"name": "mergely",
|
||||
"version": "5.3.6",
|
||||
"version": "5.4.5",
|
||||
"license": "(GPL-3.0 OR LGPL-3.0 OR MPL-1.1 OR SEE LICENSE IN LICENSE)",
|
||||
"devDependencies": {
|
||||
"@babel/core": "^7.1.6",
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "mergely",
|
||||
"version": "5.3.6",
|
||||
"version": "5.4.5",
|
||||
"description": "A javascript UI for diff/merge",
|
||||
"license": "(GPL-3.0 OR LGPL-3.0 OR MPL-1.1 OR SEE LICENSE IN LICENSE)",
|
||||
"author": {
|
||||
|
||||
@@ -18,9 +18,11 @@ function CodeMirrorDiffView(el, options) {
|
||||
const coords = this.cursorCoords(null, 'local');
|
||||
this.scrollTo(null,
|
||||
(coords.top + coords.bottom) / 2 - (this.getScrollerElement().clientHeight / 2));
|
||||
});
|
||||
this.init(el, options);
|
||||
};
|
||||
});
|
||||
this.init(el, options);
|
||||
};
|
||||
|
||||
CodeMirrorDiffView.CodeMirror = CodeMirror;
|
||||
|
||||
const trace = console.log;
|
||||
const traceTimeStart = console.time;
|
||||
@@ -316,7 +318,7 @@ CodeMirrorDiffView.prototype.bind = function(container) {
|
||||
if (!notice) {
|
||||
notice = noticeTypes.lgpl;
|
||||
}
|
||||
const editor = this._queryElement(`#${this.id}`);
|
||||
const editor = this.el;
|
||||
const splash = dom.getSplash({
|
||||
notice,
|
||||
left: (editor.offsetWidth - 300) / 2,
|
||||
@@ -738,6 +740,10 @@ CodeMirrorDiffView.prototype._isChangeInView = function(side, vp, change) {
|
||||
return true;
|
||||
}
|
||||
|
||||
// there are 3 conditions to test
|
||||
// 1: the change "from" is within the viewport from/to
|
||||
// 2: the change "to" is within the viewport from/to
|
||||
// 3: the change is so big that the viewport is within
|
||||
return (change[`${side}-line-from`] >= vp.from && change[`${side}-line-from`] <= vp.to) ||
|
||||
(change[`${side}-line-to`] >= vp.from && change[`${side}-line-to`] <= vp.to) ||
|
||||
(vp.from >= change[`${side}-line-from`] && vp.to <= change[`${side}-line-to`]);
|
||||
@@ -1088,8 +1094,8 @@ CodeMirrorDiffView.prototype._renderDiff = function(changes) {
|
||||
ctx_lhs.strokeRect(1.5, mkr_lhs_y_start, 4.5, Math.max(mkr_lhs_y_end - mkr_lhs_y_start, 5));
|
||||
ctx_lhs.stroke();
|
||||
|
||||
const mkr_rhs_y_start = change['rhs-y-start'] * lratio;
|
||||
const mkr_rhs_y_end = Math.max(change['rhs-y-end'] * lratio, 5);
|
||||
const mkr_rhs_y_start = change['rhs-y-start'] * rratio;
|
||||
const mkr_rhs_y_end = Math.max(change['rhs-y-end'] * rratio, 5);
|
||||
ctx_rhs.beginPath();
|
||||
ctx_rhs.fillStyle = '#a3a3a3';
|
||||
ctx_rhs.strokeStyle = '#000';
|
||||
@@ -1198,7 +1204,7 @@ CodeMirrorDiffView.prototype._renderDiff = function(changes) {
|
||||
|
||||
CodeMirrorDiffView.prototype._queryElement = function(selector) {
|
||||
const cacheName = `_element:${selector}`;
|
||||
const element = this[cacheName] || document.querySelector(selector);
|
||||
const element = this[cacheName] || this.el.querySelector(selector);
|
||||
if (!this[cacheName]) {
|
||||
this[cacheName] = element;
|
||||
}
|
||||
|
||||
@@ -253,6 +253,8 @@ class Mergely {
|
||||
}
|
||||
}
|
||||
|
||||
Mergely.CodeMirror = CodeMirrorDiffView.CodeMirror;
|
||||
|
||||
window.Mergely = Mergely;
|
||||
|
||||
module.exports = Mergely;
|
||||
|
||||
23
src/vdoc.js
23
src/vdoc.js
@@ -205,9 +205,13 @@ class VDoc {
|
||||
if (this.options._debug) {
|
||||
trace('vdoc#update', side, editor, viewport);
|
||||
}
|
||||
const lines = Object.keys(this._lines[side]);
|
||||
for (let i = 0; i < lines.length; ++i) {
|
||||
const id = lines[i];
|
||||
const keys = Object.keys(this._lines[side]);
|
||||
// while Mergely diffs unicode diacritic chars (letters+mark),
|
||||
// CM is by character, so diffs need to be mapped.
|
||||
const mappedChars = mapLettersToChars(editor.getValue());
|
||||
|
||||
for (const key of keys) {
|
||||
const { id } = this._lines[side][key];
|
||||
if (id < viewport.from || id > viewport.to) {
|
||||
continue;
|
||||
}
|
||||
@@ -216,7 +220,7 @@ class VDoc {
|
||||
if (vline.rendered) {
|
||||
continue;
|
||||
}
|
||||
vline.update(editor);
|
||||
vline.update(editor, mappedChars);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -257,7 +261,7 @@ class VLine {
|
||||
this.markup.push([ charFrom, charTo, className ]);
|
||||
}
|
||||
|
||||
update(editor) {
|
||||
update(editor, mappedChars) {
|
||||
if (this.rendered) {
|
||||
// FIXME: probably do not need this now
|
||||
console.log('already rendered', this.id);
|
||||
@@ -279,18 +283,17 @@ class VLine {
|
||||
editor.setGutterMarker(this.id, name, item);
|
||||
}
|
||||
if (this.markup.length) {
|
||||
// while Mergely diffs unicode chars (letters+mark), CM is by character,
|
||||
// so diffs need to be mapped.
|
||||
const mapped = mapLettersToChars(editor.getValue());
|
||||
// while Mergely diffs unicode diacritic chars (letters+mark),
|
||||
// CM is by character, so diffs need to be mapped.
|
||||
for (const markup of this.markup) {
|
||||
const [ charFrom, charTo, className ] = markup;
|
||||
const fromPos = { line: this.id };
|
||||
const toPos = { line: this.id };
|
||||
if (charFrom >= 0) {
|
||||
fromPos.ch = mapped[charFrom];
|
||||
fromPos.ch = mappedChars[charFrom];
|
||||
}
|
||||
if (charTo >= 0) {
|
||||
toPos.ch = mapped[charTo];
|
||||
toPos.ch = mappedChars[charTo];
|
||||
}
|
||||
this._clearMarkup.push(
|
||||
editor.markText(fromPos, toPos, { className }));
|
||||
|
||||
Reference in New Issue
Block a user