Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature note component #538

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Feature note component #538

wants to merge 23 commits into from

Conversation

miltonbsn
Copy link
Member

@miltonbsn miltonbsn commented May 11, 2024

closes: #433
closes: #369
closes: #457

image

image

@miltonbsn miltonbsn changed the title Start note component definition Feature note component May 11, 2024
miltonbsn and others added 10 commits May 12, 2024 15:15
- Allow more flexibility to users when writing a note
- Define min-height
- Update label text
- Match label for="" value with unique textarea id=""
- Add missing copies
- Use single quote instead of double quote on translation files
Use inset on type="radio" causes issues on Firefox
Due to that I'm forcing width and height manualy
Just to make it easy in case we need to use HEX values instead of HSL
@miltonbsn miltonbsn marked this pull request as ready for review August 18, 2024 19:20
@brmodeloweb brmodeloweb deleted a comment from allcontributors bot Aug 18, 2024
@brmodeloweb brmodeloweb deleted a comment from allcontributors bot Aug 18, 2024
@brmodeloweb brmodeloweb deleted a comment from allcontributors bot Aug 18, 2024
@brmodeloweb brmodeloweb deleted a comment from allcontributors bot Aug 18, 2024
@brmodeloweb brmodeloweb deleted a comment from allcontributors bot Aug 18, 2024
@brmodeloweb brmodeloweb deleted a comment from allcontributors bot Aug 18, 2024
@idmarjr
Copy link
Collaborator

idmarjr commented Aug 22, 2024

I did a quick manual test and found some critical bugs 😞

Auto resize note on long text strings

note-resize.mp4

Resize note issues

long-text-string-resize.mp4
note-resize-2.mp4

Select element issues after resize

resize-select-issues.mp4

@@ -566,6 +572,7 @@ const controller = function (ModelAPI, $stateParams, $rootScope, $timeout, $uibM
if(error.status == 404 || error.status == 401) {
$state.go("noaccess");
}
console.log(error);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this log.

Comment on lines +150 to +165
case "Entity-Entity":
this.connectEntities(source, target, link);
break;
case "Entity-Relationship":
this.connectEntityRelationship(target, link);
break;
case "Entity-Extension":
console.log(`conecting Entity-Extension`);
break;
case "Entity-Attribute":
console.log(`conecting Entity-Attribute`);
break;
case "Attribute-Attribute":
this.connectAttributeAttribute(source, target);
break;
case "Note-Any":

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use consts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants