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

Fork Sync Branch 16.0 #91

Merged
merged 14 commits into from
Oct 3, 2024
Merged

Fork Sync Branch 16.0 #91

merged 14 commits into from
Oct 3, 2024

Conversation

btl-bot
Copy link

@btl-bot btl-bot commented Oct 3, 2024

No description provided.

houzefa-abba and others added 14 commits April 8, 2024 09:22
Inverted dates make no functional sense, but when they do happen, this
change allows displaying them as single points in the timeline (same as
when begin=end).

We currently do have such demo data when installing project_timeline.
This commit fixes redraw issues when setting a `mode` attribute in the
`timeline` view tag.

This mode specifies a default scale one would want to set; same as when
clicking on Day/Week/Month buttons at the top of the view.

Initial rendering had issues here because data was loaded too soon,
before the timeline component was rendered/ready. The fix is to load
data into the component only after initial redraw event, called
`changed` (see <https://visjs.github.io/vis-timeline/docs/timeline/#Events>).

There was old code attempting to call `on_scale_xxx_clicked` methods at
load time to simulate clicks on these Day/Week/Month buttons, but these
methods have been renamed so this code is no longer working.

This commit also removes the `current_window` instance variable, not
needed and actually confusing as the timeline component already
maintains its own start/end information (which we can query with
`timeline.getWindow()`).
Signed-off-by pedrobaeza
Signed-off-by pedrobaeza
In addition to security rights (was already implemented), now follow
`create="0"` / `edit="0"` / `delete="0"` attributes one can set onto the
`timeline` tag, same as in other Odoo views.
Currently translated at 100.0% (18 of 18 strings)

Translation: web-16.0/web-16.0-web_editor_class_selector
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_editor_class_selector/it/
Signed-off-by pedrobaeza
…s not present

Go to the website
Open the editor
Add text and select it, a traceback is shown
Signed-off-by pedrobaeza
Copy link

@sireko sireko left a comment

Choose a reason for hiding this comment

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

Auto approved

@btl-bot btl-bot merged commit 5940ac6 into BTLzdravtech:16.0 Oct 3, 2024
6 of 8 checks passed
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.

6 participants