Skip to content

Text edition should have a clear exit option (see #371)#434

Open
Samellaperes wants to merge 2 commits into
mainfrom
improvement-371
Open

Text edition should have a clear exit option (see #371)#434
Samellaperes wants to merge 2 commits into
mainfrom
improvement-371

Conversation

@Samellaperes

Copy link
Copy Markdown

We, Samella LEUKOUO, FOTSO Vanelle, Othmane Houasli, hereby grant to Hyperglosae maintainers the right to publish our contribution under the terms of any licenses the Free Software Foundation classifies as Free Software Licenses.

@Samellaperes Samellaperes requested a review from benel as a code owner June 2, 2026 15:26
Co-authored-by: Othmane Houasli <othmane.houasli@utt.fr>
@Samellaperes

Copy link
Copy Markdown
Author

Comme nous n'avons pas eu besoin de modifier le scénario, aucune modification n'étais nécessaire au niveau des tests. Donc nous avons eu uniquement a modifier le editable.js pour rajouter un bouton. Aucune autre fonctionnalité n'a eu besoin d'être modifier.

@benel benel changed the title IMPROVEMENT: Text edition should have a clear exit option (see #371) Text edition should have a clear exit option (see #371) Jun 14, 2026

@benel benel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you @Samellaperes and @othmaneutt for your contribution.
Here are a few changes to bring to your files.

<button
type="button"
className="btn btn-success btn-sm mt-1"
onMouseDown={handleBlur}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

By firing up this event, handleBlur got invoked twice.
This generates a conflict when you edit and save several times in a row.

Comment thread frontend/src/components/EditableText.jsx Outdated
@benel benel force-pushed the main branch 3 times, most recently from 69f00a6 to 5c4b1c8 Compare June 15, 2026 13:10
@Samellaperes Samellaperes force-pushed the improvement-371 branch 2 times, most recently from 77ccd49 to 9341d11 Compare June 17, 2026 18:30
Co-authored-by: othmaneutt <othmane.houasli@utt.fr>
Co-authored-by: vanellefotso21-crypto <vanellefotso21@gmail.com>

@Samellaperes Samellaperes left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

corrections done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants