-
Notifications
You must be signed in to change notification settings - Fork 0
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
fix/refs T33256): Break segments overflow #1528
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
salisdemos
added
type:bug
Something isn't working
review:frontend
PRs that are missing review(s) from a FE perspective
labels
Jun 14, 2023
spiess-demos
approved these changes
Jun 14, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm - code review only but i seen it working.
graupnerdemos
added a commit
that referenced
this pull request
Jun 27, 2023
Squashed commit of the following: commit f5d366ebc8b3a1f1e5311d758da5c3c49fea1fae Author: graupnerdemos <graupner@demos-deutschland.de> Date: Mon Jun 19 17:51:16 2023 +0200 fix: Allow complete permission evaluation without valid user (#1565) We cannot evaluate the permissions via the db without an existing user and need to revert to evaluating directly with edt commit 8b948b3a2a3d0453c130a0cba5955d7f90e3d4e7 Author: graupnerdemos <graupner@demos-deutschland.de> Date: Mon Jun 19 16:33:21 2023 +0200 feat(refs T33234): Add rudimentary support for exposing addon permissions (#1564) Addon permissions are not configured in the classic array way the core permissions are evaluated in. Thus, all addon permissions, while having been marked as exposed, were dropped when building the frontend permission list. This change is only a rudimentary solution to exposing the addon permissions as the correct way would be to redo the permission checking in the frontend to include addon names and eventually use something more akin to hasPermission(name, context) with the context being core or an addon name commit 24e0728b0a10724161f8e7734da93227ec856205 Author: graupnerdemos <graupner@demos-deutschland.de> Date: Mon Jun 19 11:57:47 2023 +0200 fix(refs T33234): Remove incorrect late permission evaluation (#1562) The addon hook configurations do not yet provide permission information. If we were to evaluate those, we should filter the scripts in the RPC method instead of loading all scripts on the users' machine and evaluating on the client side. In addition, previously no import tabs would have been loadable from addons at all if the excel import were disabled. commit 20a4775ba44d993d47030fd227762dbc81acb486 Author: bakridemos <102021913+bakridemos@users.noreply.github.com> Date: Fri Jun 16 15:48:57 2023 +0200 fix adjust demosplan addon version - some changes made in the 0.0.4 are required. - adjust composer.lock Signed-off-by: Hamza Bakri <bakri@demos-deutschland.de> commit ba2a3fc0624088822180994f69715bd20b669682 Author: salisdemos <40487461+salisdemos@users.noreply.github.com> Date: Wed Jun 14 18:15:33 2023 +0200 chore: pick sublte button (#1535) * feat(refs T21517): Adapt DpButton usage to new dpui version demosplan-ui 0.1.3 introduces a new button variant, "subtle". As the styles for demosplan-ui are currently implememted in demosplan-core, also the new variant is implemented here. Since demosplan-ui 0.1.3 drops slots, these places have to be adapted to use icons instead of slots. All needed icons are already implemented in demosplan-ui. When icons come with a proportion signifier class, we can further tweak spacing of icons inside buttons. this will eventually be released later on. (cherry picked from commit d427b94) * chore: Bump demosplan-ui to 0.1.4 * Update package.json --------- Co-authored-by: spiess-demos <40452344+spiess-demos@users.noreply.github.com> commit 03225dcc776ef7966149108504cb717ab4ee6450 Author: graupnerdemos <graupner@demos-deutschland.de> Date: Wed Jun 14 14:36:18 2023 +0200 fix (refs T33142): uses parent statement submit date (#1530) If the entity is a segment, it now uses the submit date from the parent statement and not its own date. Co-authored-by: AlexMenschoid <meyer-schlichtmann@demos-deutschland.de> commit 3f2a89f47b683a1d7e71880df025b56ca50c0c87 Author: salisdemos <40487461+salisdemos@users.noreply.github.com> Date: Wed Jun 14 13:05:33 2023 +0200 fix/refs T33256): Break segments overflow (#1528) Add overflow-word-break to segments text boxes commit fd7a721f5161e85df13480f0530a25bbcb156491 Author: Steffi <101879352+gruenbergerdemos@users.noreply.github.com> Date: Wed Jun 14 11:17:50 2023 +0200 feat(refs T33153): Revert custom label for multiselect (#1505) * feat(refs T33153): Revert custom label for multiselect Vue multiselect uses here option and not props. commit ad98d0328cdbdb5924f38615ef436c0153124975 Author: spiess-demos <40452344+spiess-demos@users.noreply.github.com> Date: Wed Jun 14 10:57:52 2023 +0200 feat(refs T33153): Bump demosplan-ui plus some cherrypicks (#1524) bump demosplan-ui and minor fix: remove fullscreen margin + button padding (#1507) commit 4f550faca6f9c8f586a9f04e6bfc54c5615e8830 Author: Alexander Marten <marten@demos-deutschland.de> Date: Tue Jun 13 16:37:57 2023 +0200 fix (refs T33109): Flush added similar statement submitter on create new statement. commit ebf7903824f504f3c4c862537a2d81e5bcb4da31 Author: AlexMenschoid <meyer-schlichtmann@demos-deutschland.de> Date: Thu Jun 8 15:43:47 2023 +0200 fix (refs T33142): uses parent statement submit date If the entity is a segment, it now uses the submit date from the parent statement and not its own date. commit 281841ae9e0c9fbce0c2844da62999451b29fdb3 Author: Demos-CI <devops-system+github@demos-deutschland.de> Date: Tue Jun 13 11:18:42 2023 +0000 style: Apply php-cs-fixer commit b6d761eeb5448e8d52baa0c3b7015973c304184f Author: Alexander Marten <marten@demos-deutschland.de> Date: Tue Jun 13 13:13:24 2023 +0200 fix (refs T33142): Add submitDate and authoredDate to export of statements/segments if given. - only add rows in case of content is given. - remove unused translation - rename variables to enhance readability commit f7e5a282954163e672e8e305b9a8cefe9e2b1015 Author: Alexander Marten <marten@demos-deutschland.de> Date: Mon Jun 12 15:06:35 2023 +0200 fix (refs T33144): Avoid interpreting a 0 as timestamp. Actually authored date can be null/empty. Export this that way by ensure getting string date in both cases by already existing method and make another error prone converting obsolete.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
review:frontend
PRs that are missing review(s) from a FE perspective
type:bug
Something isn't working
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ticket: https://yaits.demos-deutschland.de/T33256
In the segments-list and the "erwiderungen-Ansicht" the Layout should stay as expected even if the content of the segment has looong words.
How to review/test
Linked PRs (optional)
Tasks (optional)
PR Checklist
Delete the checkbox if it doesn't apply/isn't necessary.