feat: Escape du menu slash retire le « / » + bouton « commenter » à droite du bloc (v7.66.0)
- SM.escClose() : un seul implément, appelé par les deux chemins de fermeture du menu slash (focus dans le bloc → onKd, focus dans l'entrée du menu → listener document). Retire le « / » d'ouverture seulement, remet le caret, sync() + autoSave(). - Bouton .block-comment-btn à droite du bloc (gouttière 64px) : icône bulle de conversation + , visible au survol/focus comme le handle, masqué sur mobile (2 breakpoints) et sur page verrouillée ; onclick = E.commentBlock(idx). Remplace l'ancien .block-actions de renderBlock (opacity:0 sans règle de survol → un + invisible qui capait les clics dans la gouttière). - fmtComment() restaure la plage de la toolbar (_fmtRestore) avant d'ancrer : commentBlock() ne se replie plus sur this._sel (plage périmée éventuelle). Tests: pytest tests/ -q -n auto → 1386 passed ; ruff check app tests → OK ; e2e v766 (Esc → menu fermé + bloc vide ; bouton masqué → visible au survol, à droite du wrapper, ancre = bloc visé) + v764/v762/v7641/v763/ regression_editor_mount → 7 passed sur :8081.
This commit is contained in:
@@ -1,5 +1,39 @@
|
||||
# Changelog - FlowDeck
|
||||
|
||||
## v7.66.0 (2026-10-08) — Escape du menu slash retire le « / » + bouton « commenter » à droite du bloc
|
||||
|
||||
### Added
|
||||
|
||||
- **Bouton « commenter ce bloc » à droite du bloc** (`.block-comment-btn`,
|
||||
gouttière 64px, visible au survol/focus comme le handle) : icône **bulle de
|
||||
conversation avec un +**, `onclick="E.commentBlock(idx)"` → tiroir de
|
||||
commentaires ancré sur le bloc (ou sur la sélection vivante s'il y en a une).
|
||||
Masqué sur mobile (2 breakpoints, comme le handle) et sur page verrouillée.
|
||||
- remplace l'ancien `.block-actions` de `renderBlock` : rendu en `opacity:0`
|
||||
**sans règle de survol** (jamais de classe `.visible`) → un `+` invisible
|
||||
qui capait les clics dans la gouttière droite.
|
||||
|
||||
### Fixed
|
||||
|
||||
- **Escape ferme le menu slash ET retire le « / »** : nouvelle `SM.escClose()`
|
||||
appelée par les **deux** chemins de fermeture (focus dans le bloc → `onKd`,
|
||||
focus dans l'entrée du menu → listener `document`). Ne touche au caractère
|
||||
que si c'est le `/` d'ouverture, remet le caret et sauvegarde (`sync` +
|
||||
`autoSave`).
|
||||
- **`fmtComment()` restaure la sélection de la toolbar** (`_fmtRestore()`)
|
||||
avant d'ancrer : `commentBlock()` ne se replie plus sur `this._sel` (plage
|
||||
potentiellement périmée reprise par le bouton de droite / le menu de bloc).
|
||||
|
||||
### Tests
|
||||
|
||||
- **e2e** `e2e/v766_slash_esc_comment_btn.spec.js` : « / » → menu visible →
|
||||
Escape → menu fermé et bloc vide ; bouton présent, `opacity` 0 → 1 au
|
||||
survol, **à droite** du wrapper (boîte mesurée), clic → tiroir → ancre = le
|
||||
bloc entier.
|
||||
- `tests/test_v766_slash_esc_comment_btn.py` (3) ; `pytest tests/ -q -n auto`
|
||||
→ **1386 passed**, `ruff check app tests` → OK.
|
||||
|
||||
|
||||
## v7.65.0 (2026-10-08) — Comment ancré sur la SÉLECTION de texte (pas le bloc) + retrait du bouton bleu « 💬 Comment »
|
||||
|
||||
### Changed
|
||||
|
||||
+19
@@ -1482,6 +1482,25 @@ Détails livrés :
|
||||
reload, `#_commentSelBtn` absent ; `pytest` 1383 passed, `ruff` OK ;
|
||||
`VERSION` + `app/main.py` = 7.65.0
|
||||
|
||||
### v7.66.0 — Escape du menu slash + bouton commenter à droite ✅ (2026-10-08)
|
||||
> **Objectif** : Esc efface le `/` qui a ouvert le menu slash, et chaque bloc
|
||||
> gagne un bouton de commentaire à droite. **COMPLETED**.
|
||||
|
||||
- [x] **`SM.escClose()`** — un seul implément, appelé par les deux chemins
|
||||
(focus dans le bloc → `onKd`, focus dans l'entrée du menu → listener
|
||||
`document`) ; retire le `/` d'ouverture seulement, remet le caret, `sync` +
|
||||
`autoSave`
|
||||
- [x] **Bouton à droite du bloc** — `.block-comment-btn` (bulle + `+`,
|
||||
`E.commentBlock(idx)`), gouttière 64px, visible au survol, masqué mobile +
|
||||
page verrouillée ; remplace l'ancien `.block-actions` mort (opacity:0 sans
|
||||
survol → `+` invisible qui capait les clics)
|
||||
- [x] **`fmtComment()`** restaure la plage de la toolbar avant d'ancrer ;
|
||||
`commentBlock()` ne se replie plus sur `this._sel` périmé
|
||||
- [x] **Tests** — e2e `e2e/v766_slash_esc_comment_btn.spec.js` (Esc → menu
|
||||
fermé + bloc vide ; bouton à droite mesuré, ancre = bloc entier),
|
||||
`tests/test_v766_slash_esc_comment_btn.py` (3) ; `pytest` **1386 passed**,
|
||||
`ruff` OK ; `VERSION` + `app/main.py` = 7.66.0
|
||||
|
||||
---
|
||||
|
||||
## ✅ Fonctionnalités livrées hors roadmap (bonus détectés dans le code)
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
# WORKLOAD — FlowDeck Notion Clone
|
||||
|
||||
> **Début**: 2026-07-08 | **Version**: v7.65.0 (comment ancré sur la sélection de texte, bouton bleu retiré) | **Statut**: EN COURS 🔄
|
||||
> **Début**: 2026-07-08 | **Version**: v7.66.0 (Escape du menu slash retire le / + bouton commenter à droite du bloc) | **Statut**: EN COURS 🔄
|
||||
> **Cible**: parité Notion + intégration forge · **Follow-ups v7.3 livrés**: sidebar teamspaces, notif `page.updated`, charts `number` + dashboards multi-DB, unfurl forge, UI Settings → Audit — voir `ROADMAP.md § v7.3.0`
|
||||
|
||||
## Avancement Global
|
||||
|
||||
+1
-1
@@ -186,7 +186,7 @@ async def lifespan(_app: FastAPI):
|
||||
|
||||
app = FastAPI(
|
||||
title="FlowDeck",
|
||||
version="7.65.0",
|
||||
version="7.66.0",
|
||||
docs_url="/docs",
|
||||
redoc_url="/redoc",
|
||||
lifespan=lifespan,
|
||||
|
||||
@@ -0,0 +1,18 @@
|
||||
feat: Escape du menu slash retire le « / » + bouton « commenter » à droite du bloc (v7.66.0)
|
||||
|
||||
- SM.escClose() : un seul implément, appelé par les deux chemins de fermeture
|
||||
du menu slash (focus dans le bloc → onKd, focus dans l'entrée du menu →
|
||||
listener document). Retire le « / » d'ouverture seulement, remet le caret,
|
||||
sync() + autoSave().
|
||||
- Bouton .block-comment-btn à droite du bloc (gouttière 64px) : icône bulle de
|
||||
conversation + , visible au survol/focus comme le handle, masqué sur mobile
|
||||
(2 breakpoints) et sur page verrouillée ; onclick = E.commentBlock(idx).
|
||||
Remplace l'ancien .block-actions de renderBlock (opacity:0 sans règle de
|
||||
survol → un + invisible qui capait les clics dans la gouttière).
|
||||
- fmtComment() restaure la plage de la toolbar (_fmtRestore) avant d'ancrer :
|
||||
commentBlock() ne se replie plus sur this._sel (plage périmée éventuelle).
|
||||
|
||||
Tests: pytest tests/ -q -n auto → 1386 passed ; ruff check app tests → OK ;
|
||||
e2e v766 (Esc → menu fermé + bloc vide ; bouton masqué → visible au survol,
|
||||
à droite du wrapper, ancre = bloc visé) + v764/v762/v7641/v763/
|
||||
regression_editor_mount → 7 passed sur :8081.
|
||||
@@ -2,7 +2,7 @@
|
||||
"openapi": "3.1.0",
|
||||
"info": {
|
||||
"title": "FlowDeck",
|
||||
"version": "7.65.0"
|
||||
"version": "7.66.0"
|
||||
},
|
||||
"paths": {
|
||||
"/auth/register": {
|
||||
|
||||
@@ -0,0 +1,99 @@
|
||||
const { test, expect } = require('@playwright/test');
|
||||
|
||||
/**
|
||||
* v7.66 — deux ajustements éditeur :
|
||||
* - « / » ouvre le menu slash → Escape ferme le menu ET retire le « / »
|
||||
* - survol d'un bloc → bouton à droite (bulle + ) qui commente le bloc
|
||||
* Instance de test attendue sur FD_BASE_URL (défaut 8081).
|
||||
*/
|
||||
const FD_BASE = process.env.FD_BASE_URL || 'http://localhost:8081';
|
||||
const USER = process.env.FD_USER || '[email protected]';
|
||||
const PASS = process.env.FD_PASS || 'e2e-secret-123';
|
||||
|
||||
test.use({ serviceWorkers: 'block' });
|
||||
|
||||
async function login(page) {
|
||||
await page.goto(`${FD_BASE}/auth/login?provider=local`, { waitUntil: 'domcontentloaded' });
|
||||
await page.fill('#email', USER);
|
||||
await page.fill('#password', PASS);
|
||||
await page.click('.btn-primary');
|
||||
const ok = await page
|
||||
.waitForURL('**/workspaces', { timeout: 8000 })
|
||||
.then(() => true)
|
||||
.catch(() => false);
|
||||
if (!ok) {
|
||||
const resp = await page.request.post(`${FD_BASE}/auth/register`, {
|
||||
data: { email: USER, password: PASS, name: 'E2E' },
|
||||
});
|
||||
if (resp.status() !== 409) expect(resp.ok()).toBeTruthy();
|
||||
await page.goto(`${FD_BASE}/workspaces`, { waitUntil: 'domcontentloaded' });
|
||||
}
|
||||
}
|
||||
|
||||
async function newPage(page, body) {
|
||||
const cookies = await page.context().cookies();
|
||||
const csrf = (cookies.find((c) => c.name === 'csrf_token') || {}).value || '';
|
||||
const created = await page.request.post(
|
||||
`${FD_BASE}/board/api/pages?title=v766 ${Date.now()}`,
|
||||
{ headers: { 'X-CSRF-Token': csrf } }
|
||||
);
|
||||
expect(created.status(), await created.text()).toBeLessThan(400);
|
||||
const { id } = await created.json();
|
||||
if (body) {
|
||||
const bs = await page.request.post(`${FD_BASE}/board/api/pages/${id}/blocks`, {
|
||||
headers: { 'X-CSRF-Token': csrf },
|
||||
data: { title: 'v766', blocks: body },
|
||||
});
|
||||
expect(bs.status(), await bs.text()).toBeLessThan(400);
|
||||
}
|
||||
await page.goto(`${FD_BASE}/pages/${id}`, { waitUntil: 'domcontentloaded' });
|
||||
await page.waitForSelector('#_blocksCt [data-bid]', { timeout: 15000 });
|
||||
return id;
|
||||
}
|
||||
|
||||
test('Escape du menu slash retire le caractère /', async ({ page }) => {
|
||||
await login(page);
|
||||
await newPage(page, [{ id: 'b1', type: 'paragraph', content: '' }]);
|
||||
const blk = page.locator('#_blocksCt [data-bid]').first();
|
||||
await blk.click();
|
||||
await page.keyboard.type('/');
|
||||
await expect(page.locator('#_slashMenu')).toBeVisible({ timeout: 5000 });
|
||||
await page.keyboard.press('Escape');
|
||||
await expect(page.locator('#_slashMenu')).toBeHidden();
|
||||
// le « / » a été retiré — pas de caractère résiduel
|
||||
await expect(blk).toHaveText('');
|
||||
await page.screenshot({ path: 'shots/v766-slash-esc.png' });
|
||||
});
|
||||
|
||||
test('bouton commenter à droite du bloc', async ({ page }) => {
|
||||
await login(page);
|
||||
// 2 blocs : l'éditeur focalise le 1er au chargement (`:focus-within` le
|
||||
// montre) → on mesure sur le 2e, ni focalisé ni survolé.
|
||||
await newPage(page, [
|
||||
{ id: 'b1', type: 'paragraph', content: 'Premier bloc' },
|
||||
{ id: 'b2', type: 'paragraph', content: 'Un bloc a commenter' },
|
||||
]);
|
||||
await expect(page.locator('#_blocksCt .block-comment-btn')).toHaveCount(2);
|
||||
const wrap = page.locator('#_blocksCt .block-wrapper').nth(1);
|
||||
const btn = page.locator('#_blocksCt .block-comment-btn').nth(1);
|
||||
|
||||
// masqué tant qu'on ne survole pas / ne focalise pas le bloc
|
||||
await expect.poll(() => btn.evaluate((el) => getComputedStyle(el).opacity)).toBe('0');
|
||||
await wrap.hover();
|
||||
await expect.poll(() => btn.evaluate((el) => getComputedStyle(el).opacity)).toBe('1');
|
||||
|
||||
// bien à droite du bloc (gouttière 64px)
|
||||
const wb = await wrap.boundingBox();
|
||||
const bb = await btn.boundingBox();
|
||||
expect(bb.x).toBeGreaterThan(wb.x + wb.width - 1);
|
||||
await page.screenshot({ path: 'shots/v766-comment-btn.png' });
|
||||
|
||||
// clic → tiroir → envoi → ancre = le bloc visé (et pas le 1er)
|
||||
await btn.click();
|
||||
const drawer = page.locator('.comments-drawer');
|
||||
await expect(drawer).toBeVisible();
|
||||
await drawer.locator('textarea').fill('Commentaire via le bouton de droite');
|
||||
await drawer.locator('.ct-send').click();
|
||||
await expect(page.locator('.comment-thread')).toHaveCount(1);
|
||||
await expect(page.locator('#_blocksCt .fd-anchor[data-cid]')).toHaveText('Un bloc a commenter');
|
||||
});
|
||||
@@ -2371,6 +2371,35 @@ button.breadcrumb-current { max-width: 260px; }
|
||||
color: var(--text-primary);
|
||||
}
|
||||
|
||||
/* ── v7.66 : bouton « commenter ce bloc » — à droite du bloc (gouttière 64px),
|
||||
bulle de conversation + , visible au survol comme le handle ── */
|
||||
.block-comment-btn {
|
||||
position: absolute;
|
||||
right: -60px;
|
||||
top: 4px;
|
||||
width: 22px;
|
||||
height: 22px;
|
||||
padding: 0;
|
||||
display: flex;
|
||||
align-items: center;
|
||||
justify-content: center;
|
||||
background: none;
|
||||
border: none;
|
||||
border-radius: 3px;
|
||||
color: var(--text-dim);
|
||||
cursor: pointer;
|
||||
opacity: 0;
|
||||
transition: opacity 0.15s ease;
|
||||
}
|
||||
.block-wrapper:hover .block-comment-btn,
|
||||
.block-wrapper:focus-within .block-comment-btn {
|
||||
opacity: 1;
|
||||
}
|
||||
.block-comment-btn:hover {
|
||||
background: var(--bg-hover);
|
||||
color: var(--text-primary);
|
||||
}
|
||||
|
||||
/* ── v5.10.0: selection multiple + menu contexte bloc ── */
|
||||
.block-wrapper.selected {
|
||||
background: rgba(35,131,226,0.06);
|
||||
@@ -3288,6 +3317,7 @@ button.breadcrumb-current { max-width: 260px; }
|
||||
.block-handle { display: none; }
|
||||
.block-insert-btn { display: none; }
|
||||
.block-actions { display: none; }
|
||||
.block-comment-btn { display: none; }
|
||||
.editor-statusbar { left: 0; }
|
||||
}
|
||||
|
||||
@@ -3855,6 +3885,7 @@ button.breadcrumb-current { max-width: 260px; }
|
||||
.block-handle { display: none; }
|
||||
.block-insert-btn { display: none; }
|
||||
.block-actions { display: none; }
|
||||
.block-comment-btn { display: none; }
|
||||
.page-title-block::before { display: none; }
|
||||
.page-title-block .page-icon-emoji { margin-left: 0; }
|
||||
.page-title-block .page-title-input { padding-left: 0; }
|
||||
@@ -4998,6 +5029,7 @@ a.fd-wiki-link:hover { border-bottom-color: var(--accent, #2383e2); }
|
||||
font-weight: 500;
|
||||
}
|
||||
body.page-locked .block-handle,
|
||||
body.page-locked .block-comment-btn,
|
||||
body.page-locked .block-add-btn,
|
||||
body.page-locked .format-toolbar,
|
||||
body.page-locked .get-started-toolbar { display: none !important; }
|
||||
|
||||
@@ -463,6 +463,27 @@ const PD=(()=>{try{const el=document.getElementById('page-data');return el?JSON.
|
||||
SM._e.style.left=Math.min(r.left,window.innerWidth-340)+'px';
|
||||
SM._rs();setTimeout(()=>SM._in.focus(),10);SM._o=true;},
|
||||
close(){if(SM._e)SM._e.style.display='none';SM._o=false;SM._in&&(SM._in.value='');if(SM._pv)SM._pv.style.display='none';MTB.close();},
|
||||
// v7.66 : Escape ferme le menu ET retire le « / » qui l'a ouvert.
|
||||
escClose(){
|
||||
const i=SM._i;SM.close();
|
||||
const E=window.E;if(!E||i<0||i>=E.blocks.length)return;
|
||||
const el=E.getEl(E.blocks[i].id);if(!el)return;
|
||||
const w=document.createTreeWalker(el,NodeFilter.SHOW_TEXT);let n;
|
||||
while((n=w.nextNode())){
|
||||
if(n.parentElement&&n.parentElement.closest('.fd-react-chip'))continue;
|
||||
const v=n.nodeValue,at=v.indexOf('/');
|
||||
if(at>=0){
|
||||
if(v.slice(0,at).trim())return; // pas le / d'ouverture → on ne touche à rien
|
||||
n.nodeValue=v.slice(0,at)+v.slice(at+1);
|
||||
E.sync();E.dirty=true;E.autoSave();
|
||||
el.focus();
|
||||
const r=document.createRange();r.setStart(n,Math.min(at,n.nodeValue.length));r.collapse(true);
|
||||
const s=window.getSelection();s.removeAllRanges();s.addRange(r);
|
||||
return;
|
||||
}
|
||||
if(v.trim())return; // 1er texte non vide sans / → rien à retirer
|
||||
}
|
||||
},
|
||||
// Ligne d'item du slash (partagée par la popup desktop et la feuille mobile)
|
||||
_slashItemHtml(c){return `<div class="slash-item" data-sid="${c.id}" style="display:flex;align-items:center;gap:10px;padding:7px 10px;font-size:14px;color:var(--text-primary);cursor:pointer;border-radius:5px">
|
||||
<span class="slash-item-icon" style="width:26px;height:26px;display:flex;align-items:center;justify-content:center;font-size:15px;background:var(--bg-secondary);border-radius:4px;flex-shrink:0">${c.icon}</span>
|
||||
@@ -476,7 +497,7 @@ const PD=(()=>{try{const el=document.getElementById('page-data');return el?JSON.
|
||||
_kd(e){if(e.key==='ArrowDown'){e.preventDefault();SM._s=Math.min(SM._s+1,SM._it.length-1);SM._rs();}
|
||||
else if(e.key==='ArrowUp'){e.preventDefault();SM._s=Math.max(SM._s-1,0);SM._rs();}
|
||||
else if(e.key==='Enter'){e.preventDefault();const c=SM._it[SM._s];if(c)SM.sel(c.id);}
|
||||
else if(e.key==='Escape'){e.preventDefault();SM.close();}},
|
||||
else if(e.key==='Escape'){e.preventDefault();SM.escClose();}},
|
||||
_rs(){const its=SM._li.querySelectorAll('.slash-item');its.forEach((el,i)=>{el.classList.toggle('selected',i===SM._s);});const cur=its[SM._s];if(cur&&cur.scrollIntoView)cur.scrollIntoView({block:'nearest'});},
|
||||
sel(id){if(window.E)window.E.applySlash(id);SM.close();},
|
||||
get isOpen(){return SM._o;}};
|
||||
@@ -789,14 +810,17 @@ const PD=(()=>{try{const el=document.getElementById('page-data');return el?JSON.
|
||||
let s=idx; while(s>0&&bs[s-1]&&bs[s-1].type==='numbered_list')s--;
|
||||
numAttr=` data-num="${idx-s+1}"`;
|
||||
}
|
||||
const actions=idx>=0?`<div class="block-actions"><button class="block-add-btn" onclick="E.addAfter(${idx})">+</button></div>`:'';
|
||||
// v7.66 : bouton « commenter ce bloc » à droite du bloc (gouttière 64px),
|
||||
// icône bulle de conversation + . Remplace l'ancien .block-actions mort
|
||||
// (opacity:0 sans règle de survol → un + invisible qui capait les clics).
|
||||
const cmtBtn=idx>=0?`<button type="button" class="block-comment-btn" title="Comment on block" aria-label="Comment on block" onclick="E.commentBlock(${idx})"><svg viewBox="0 0 16 16" width="14" height="14" fill="none" stroke="currentColor" stroke-width="1.3" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M13.6 7.3C13.6 10.2 11 12.5 7.9 12.5C7.3 12.5 6.7 12.4 6.2 12.2L3.5 13.3L4.4 10.9C3.4 10 2.4 8.7 2.4 7.3C2.4 4.4 5 2.2 8 2.2C11 2.2 13.6 4.4 13.6 7.3Z"/><path d="M8 5.3V9.3M6 7.3H10"/></svg></button>`:'';
|
||||
// + « ajouter un bloc dessous », juste avant le handle (gouttière 64px)
|
||||
const insertBtn=idx>=0?`<button type="button" class="block-insert-btn" title="Add block below" aria-label="Add block below" onclick="E.addAfter(${idx})">+</button>`:'';
|
||||
return `<div class="block-wrapper" data-id="${b.id}">
|
||||
${insertBtn}
|
||||
<div class="block-handle"><svg viewBox="0 0 16 16" width="14" height="14"><circle cx="4" cy="3" r="1.3"/><circle cx="11" cy="3" r="1.3"/><circle cx="4" cy="8" r="1.3"/><circle cx="11" cy="8" r="1.3"/><circle cx="4" cy="13" r="1.3"/><circle cx="11" cy="13" r="1.3"/></svg></div>
|
||||
${actions}
|
||||
<${tag}${style}${bidAttr}${numAttr} class="${cls}">${inner}</${tag}>
|
||||
${cmtBtn}
|
||||
</div>`;
|
||||
}
|
||||
|
||||
@@ -1865,19 +1889,16 @@ replaceBlock(idx,type){const b=this.blocks[idx];if(!b)return;this.sync();this.pu
|
||||
setBlockColor(idx,key,val){if(idx<0||idx>=this.blocks.length)return;const targets=this._blockTargets(idx);const v=(!val||val==='default')?null:val;this.sync();this.pushHistory();targets.forEach(i=>{const b=this.blocks[i];if(!b)return;if(!b.style)b.style={};if(v==null)delete b.style[key];else b.style[key]=v;});if(v!=null){try{localStorage.setItem('fdLastBlockColor',key+'|'+v);}catch(e){}}this.dirty=true;this.autoSave();this.render();_rtSync();_selClear();},
|
||||
// ── v7.61 : actions du menu contextuel de bloc ──
|
||||
commentBlock(idx){
|
||||
// L'ancre est la SÉLECTION de texte si elle vit dans ce bloc (la
|
||||
// sélection de la toolbar aussi : le clic sur le bouton peut l'avoir
|
||||
// fait disparaître du DOM), sinon tout le bloc.
|
||||
// L'ancre est la SÉLECTION de texte vivante dans ce bloc, sinon tout
|
||||
// le bloc. La toolbar de sélection restaure sa plage AVANT (fmtComment)
|
||||
// — pas de repli sur `this._sel` ici : sinon le bouton de droite / le
|
||||
// menu de bloc reprendraient une sélection périmée.
|
||||
const b=this.blocks[idx];if(!b)return;const el=this.getEl(b.id);if(!el)return;
|
||||
let r=null;const s=window.getSelection();
|
||||
if(s&&s.rangeCount&&!s.isCollapsed){
|
||||
const rr=s.getRangeAt(0);
|
||||
if(el.contains(rr.startContainer)&&el.contains(rr.endContainer))r=rr.cloneRange();
|
||||
}
|
||||
if(!r&&this._sel&&this._sel.range&&this._sel.idx===idx
|
||||
&&el.contains(this._sel.range.startContainer)&&el.contains(this._sel.range.endContainer)){
|
||||
r=this._sel.range.cloneRange();
|
||||
}
|
||||
if(!r){r=document.createRange();r.selectNodeContents(el);}
|
||||
const start=_srcOffset(el,r.startContainer,r.startOffset);
|
||||
const end=_srcOffset(el,r.endContainer,r.endOffset);
|
||||
@@ -2386,7 +2407,7 @@ applyAIBlocks(text){
|
||||
const ab=this.getActiveBlock();if(!ab)return;
|
||||
const{el,bid,idx}=ab;const block=this.blocks[idx];if(!block)return;
|
||||
if(SM.isOpen){
|
||||
if(e.key==='Escape'){e.preventDefault();SM.close();return;}
|
||||
if(e.key==='Escape'){e.preventDefault();SM.escClose();return;}
|
||||
if(e.key==='ArrowDown'||e.key==='ArrowUp'||e.key==='Enter'){e.preventDefault();return;}
|
||||
if(e.key==='Backspace'&&!SM._q&&(el.textContent||'').trim()==='/'){e.preventDefault();SM.close();return;}
|
||||
// La requête peut être tapée DANS le bloc (« /numbered list ») si le
|
||||
@@ -2594,7 +2615,7 @@ applyAIBlocks(text){
|
||||
// « … » → le menu contextuel de bloc (recherche + Text + pied métadonnées)
|
||||
fmtMore(ev){const bid=this._fmtBid();if(!bid)return;const r=ev.currentTarget.getBoundingClientRect();
|
||||
this.fmt.open=false;blockMenuOpen(Math.max(8,Math.min(r.left-190,window.innerWidth-250)),r.bottom+6,bid);},
|
||||
fmtComment(){this.fmt.open=false;const i=this.fmt.idx;if(i>=0)this.commentBlock(i);},
|
||||
fmtComment(){this.fmt.open=false;const i=this.fmt.idx;if(i<0)return;this._fmtRestore();this.commentBlock(i);},
|
||||
fmtAskAI(){this.fmt.open=false;const i=this.fmt.idx;if(i>=0)this.askAIBlock(i);},
|
||||
fmtEditAI(){this.fmtAskAI();},
|
||||
fmtSkill(tpl){this.fmt.open=false;const i=this.fmt.idx;if(i>=0)this.runSkill(i,tpl);},
|
||||
|
||||
@@ -0,0 +1,41 @@
|
||||
"""7.66.0 — Escape du menu slash retire le « / » + bouton commenter à droite du bloc."""
|
||||
|
||||
from pathlib import Path
|
||||
|
||||
ROOT = Path(__file__).resolve().parent.parent
|
||||
EDITOR_JS = (ROOT / "static" / "js" / "page_editor_scripts.js").read_text(encoding="utf-8")
|
||||
APP_CSS = (ROOT / "static" / "css" / "app.css").read_text(encoding="utf-8")
|
||||
|
||||
|
||||
def test_slash_escape_removes_the_slash():
|
||||
# une seule implémentation, appelée par les deux chemins (focus dans le bloc
|
||||
# → onKd ; focus dans l'entrée du menu → listener document)
|
||||
assert "escClose(){" in EDITOR_JS
|
||||
assert EDITOR_JS.count("SM.escClose();") == 2
|
||||
# ne retire que le « / » d'ouverture, remet le caret et sauvegarde
|
||||
assert "n.nodeValue=v.slice(0,at)+v.slice(at+1);" in EDITOR_JS
|
||||
assert "E.sync();E.dirty=true;E.autoSave();" in EDITOR_JS
|
||||
|
||||
|
||||
def test_comment_button_on_the_right_of_the_block():
|
||||
# émis APRÈS le contenu du bloc → à droite dans le flex du wrapper
|
||||
i_tpl = EDITOR_JS.index("const cmtBtn=")
|
||||
i_content = EDITOR_JS.index('class="${cls}">${inner}</${tag}>', i_tpl)
|
||||
i_btn = EDITOR_JS.index("${cmtBtn}", i_tpl)
|
||||
assert i_content < i_btn
|
||||
# icône bulle de conversation +, ancre = commentaire du bloc
|
||||
assert 'onclick="E.commentBlock(${idx})"' in EDITOR_JS
|
||||
assert EDITOR_JS.count("<path d=\"M13.6 7.3C13.6 10.2") == 1
|
||||
assert 'stroke-width="1.3"' in EDITOR_JS
|
||||
# l'ancien .block-actions mort (opacity:0 sans survol) a disparu de renderBlock
|
||||
assert "const actions=idx>=0?" not in EDITOR_JS
|
||||
|
||||
|
||||
def test_comment_button_style_and_visibility():
|
||||
assert ".block-comment-btn {" in APP_CSS
|
||||
assert "right: -60px;" in APP_CSS # gouttière 64px, comme .block-actions
|
||||
assert ".block-wrapper:hover .block-comment-btn," in APP_CSS
|
||||
assert ".block-comment-btn:hover {" in APP_CSS
|
||||
# masqué partout où le handle l'est : mobile (2 breakpoints) + page verrouillée
|
||||
assert APP_CSS.count(".block-comment-btn { display: none; }") == 2
|
||||
assert "body.page-locked .block-comment-btn," in APP_CSS
|
||||
Reference in New Issue
Block a user