Fossil

Differences From Artifact [ccf1183ab2]:

• File src/fossil.info-diff.js — part of check-in [cfdd666533] at 2021-03-01 17:35:49 on branch diff-view-toggle-poc — Proof of concept/demo for toggle buttons on individual diff views on the /info diff views, as discussed in [forum:0f751ad9c8]. (user: stephan size: 622)

To Artifact [7db676b32e]:

• File src/fossil.info-diff.js — part of check-in [f9d29b9702] at 2021-03-02 03:51:47 on branch diff-view-toggle-poc — Moved the diff toggle button to the right of the diff header line, per forum feedback. Changed the button to a checkbox. Removed the arbitrary heuristics regarding which diffs to show/hide by default - they are now all on by default. (user: stephan size: 486) [more...]

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18

"use strict";
window.fossil.onPageLoad(function(){
  const F = window.fossil, D = F.dom;
  const addToggle = function(diffElem){
    const cs = diffElem.getClientRects()[0];
    if(cs.height < 150/*arbitrary*/) return;
    const btn = D.addClass(D.button("Toggle diff view"), 'diff-toggle'),
          p = diffElem.parentElement;
    p.insertBefore(btn, diffElem);
    btn.addEventListener('click', function(){
      diffElem.classList.toggle('hidden');
    }, false);
    if(cs.height > 700/*arbitrary!*/){
      btn.click();
    }
  };
  document.querySelectorAll('pre.udiff, table.sbsdiffcols').forEach(addToggle);
});

|
<
|
|
|



<
<
<

1
2
3
4
5

6
7
8
9
10
11



12
13
14

"use strict";
window.fossil.onPageLoad(function(){
  const F = window.fossil, D = F.dom;
  const addToggle = function(diffElem){
    const sib = diffElem.previousElementSibling,

          btn = sib ? D.addClass(D.checkbox(true), 'diff-toggle') : 0;
    if(!sib) return;
    D.append(sib,btn);
    btn.addEventListener('click', function(){
      diffElem.classList.toggle('hidden');
    }, false);



  };
  document.querySelectorAll('pre.udiff, table.sbsdiffcols').forEach(addToggle);
});