Implement autoarrange collection when adding a new collection
@Ivanruii ci sta già lavorando.
Dal 26/6/2024.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Right now when you click on Add collection the collection is place on the top left corner of the visible area of the canvas, if a blind person tries to add more than one collection they will be place stacked, that's not an accesible approach.
The idea is to implement an algorithm that:
- Will find for free spots (or at least the ones with less collision area).
- Will propose a where to place the collection starting in the center of the visible area.
- Will place next tables following an spiral path.
Once the table is added we should highlight it using some animation just to let the user know where has been placed.
There's an initial POC implemented:
https://github.com/Lemoncode/auto-arrange-playground
And how it works:
https://github.com/Lemoncode/mongo-modeler/assets/1457912/447236a3-4364-427c-8428-996088c2abd6
A previous discussion is needed before starting integrating the POC, the steps:
- Understand the current solution (team start).
- Adapt it to mongo modeler as is and give it a try.
- Enhance it and add unit testing once we check that it fits with Mongo Modeler.
Steps to implement this:
-
In common under common/helper create a subfolder called
autorrange-table. -
Under that subfolder create a
autoarrange-table.model.tswe will slightly update the model (removecolorfield).
./src/common/autoarrange-table/autoarrange-table.model.ts
export interface Box {
x: number;
y: number;
width: number;
height: number;
}
- Add the util to calculate overlapping etc...
./src/common/autoarrange-table/autoarrange-table.utils.ts
import { Box } from './autoarrange-table.model';
export const getRandomInt = (min: number, max: number): number => {
return Math.floor(Math.random() * (max - min + 1)) + min;
};
export const isOverlapping = (box1: Box, box2: Box): boolean => {
return (
box1.x < box2.x + box2.width &&
box1.x + box1.width > box2.x &&
box1.y < box2.y + box2.height &&
box1.y + box1.height > box2.y
);
};
export const calculateCollisionArea = (box1: Box, box2: Box): number => {
const xOverlap = Math.max(
0,
Math.min(box1.x + box1.width, box2.x + box2.width) -
Math.max(box1.x, box2.x)
);
const yOverlap = Math.max(
0,
Math.min(box1.y + box1.height, box2.y + box2.height) -
Math.max(box1.y, box2.y)
);
return xOverlap * yOverlap;
};
And create the main file:
./src/common/autoarrange-table/index.ts
import { Box, Size } from "../model";
import { calculateCollisionArea, isOverlapping } from "./canvas.utils";
function* spiralPositions(
centerX: number,
centerY: number,
canvasSize: Size
): Generator<[number, number]> {
let x = 0,
y = 0,
dx = 0,
dy = -1;
for (let i = 0; i < Math.max(canvasSize.width, canvasSize.height) ** 2; i++) {
if (
-canvasSize.width / 2 < x &&
x < canvasSize.width / 2 &&
-canvasSize.height / 2 < y &&
y < canvasSize.height / 2
) {
yield [centerX + x, centerY + y];
}
if (x === y || (x < 0 && x === -y) || (x > 0 && x === 1 - y)) {
[dx, dy] = [-dy, dx];
}
x += dx;
y += dy;
}
}
export function findFreePositionOrMinCollision(
boxes: Box[],
newBoxSize: Size,
canvasSize: Size
): Box | null {
const centerX = Math.floor(canvasSize.width / 2);
const centerY = Math.floor(canvasSize.height / 2);
let minCollisionBox: Box | null = null;
let minCollisionArea = Infinity;
for (const [x, y] of spiralPositions(centerX, centerY, canvasSize)) {
const newBox = {
x,
y,
width: newBoxSize.width,
height: newBoxSize.height,
// TODO: we will remove this once we get rid of the poc
// and integrate this into the main app
color: "orange",
};
if (
x >= 0 &&
y >= 0 &&
x + newBoxSize.width <= canvasSize.width &&
y + newBoxSize.height <= canvasSize.height
) {
let collisionArea = 0;
let isFree = true;
for (const existingBox of boxes) {
if (isOverlapping(newBox, existingBox)) {
isFree = false;
collisionArea += calculateCollisionArea(newBox, existingBox);
}
}
if (isFree) {
return newBox;
}
if (collisionArea < minCollisionArea) {
minCollisionArea = collisionArea;
minCollisionBox = newBox;
}
}
}
if (minCollisionBox !== null) {
return minCollisionBox;
}
// TODO: if no free position is found, return a random one
return null;
}
Now on the Add table handler let's use the findFreePositionOrMinCollision, to add a new collection, things to take into consideration:
- Let's map all the tables from
TableVmtoBox(just create a mapper), to take into account we will have to calculate the fullsize of each table (witdth and height) and add a margin e.g. 40px. - Then call
findFreePositionOrMinCollisionpassing the box collection and the current view size (canvasViewSettings.scrollPosition.x,canvasViewSettings.scrollPosition.y, and the current subview with the width and height of the part that is shown. - Add some transition or animation to highlight the table temporarily once is created
Check that the approach is working, then we will do a second iteration (refactor the code, add unit testing etc...)
- Lingua principale
- TypeScript
- Stelle
- 139
- Fork
- 40
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Lemoncode/mongo-modeler
-
Add color customization to relationship lines and tablesForse già presa @RamiroRodriguezC l’ha presa 131 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
Lemoncode/mongo-modeler#610 · 2 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
Lemoncode/mongo-modeler#609 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Lemoncode/mongo-modeler#597 ·
-
[VSCode Extension] Hide Open and Save toolbar buttonsForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
Lemoncode/mongo-modeler#591 ·
-
[VSCode Extension] Integrate new file functionalityForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
Lemoncode/mongo-modeler#590 ·
Tutte le issue di Lemoncode/mongo-modeler
Issue simili
-
check:passed streams:add
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 2 giorni
-
beta technical-medium ui
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
walletbeat/walletbeat#1625 ·
I maintainer di solito rispondono entro 1 giorno
-
Good First Issue hacktoberfest
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
hiero-ledger/hiero-sdk-js#4489 ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug] The clients language filter cannot select the rows the page labels as unknownForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
apache/rocketmq-dashboard#6103 ·
I maintainer di solito rispondono entro 4 giorni
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
payloadcms/payload#18652 ·
I maintainer di solito rispondono entro 1 giorno