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

Enable edit labels #2

Closed
wants to merge 40 commits into from
Closed

Conversation

Uriel-Sautron
Copy link
Collaborator

Description

Issues

Checklist

  • This PR contains a description of the changes I'm making
  • I've read the Contribution Guidelines
  • I've added tests for changes or features I've introduced
  • I documented any high-level concepts I'm introducing in documentation/
  • CI is currently green and this is ready for review

emersion and others added 30 commits August 5, 2024 18:07
…heBundesbahnen#211)

* fix: disable notes in filter sidebar in standalone mode

Notes are disabled in standalone mode.

* If the backend is turned off, the notes - labels will no longer be displayed in the filter. To keep everything consistent, these labels will also be deactivated in the filter and hidden in the note-edit dialog.

---------

Co-authored-by: adrian_egli <[email protected]>
…Bundesbahnen/disable-backend-node-links

fix: disable links section in node sidebar in standalone mode
…ahnen#217)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ahnen#219)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
added       manifest-file: .release-please-manifest.json to release please
Release-As: 2.7.3
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* create/delete node

* remove label operation logic

* definition of all operations

* node operations

* trainrun operations

* lint

* remove unnecessary comments

* remove unnecessary param + add objectType to Operation payload

* simplify operation design for first use

* refactor operation classes with OperationObjectType

* add documentation

* remove generic

* drop generic in operation sub-classes

* rename documentation + lint
…chweizerischeBundesbahnen#228)

* Fix tag patterns complying with extended tags from release-please.

Signed-off-by: chenkins <[email protected]>

* Fix link in documentation.

Signed-off-by: chenkins <[email protected]>

---------

Signed-off-by: chenkins <[email protected]>
…desbahnen#230)

According to the sbb-angular docs [1], the sbb-header-environment
component should be disabled in production. Do so in standalone
mode.

Without this, a ribbon with a "standalone" text is displayed in
the top-left corner.

[1]: https://github.com/sbb-design-systems/sbb-angular/blob/main/src/angular/header-lean/header-lean.md#environment
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Release-As: 2.7.5
…ahnen#234)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* removed unrequired packages

* All patches updates (npx npm-check-updates -u -t patch)
Keyboard elements (buttons) will visually highlight.
…bahnen#247)

- if so, press ...
- for more details ...
- Visibility
…Bundesbahnen#245)

- ... system saves it ...
- ... Use the copy-paste idea: Select the train and press ...
…rischeBundesbahnen#248)

Fixes a few issues:

- "keyup" is not triggered when copy-pasting. So if the user right
  clicks and pastes a new value, the name would not get updated.
- "keyup" triggers each time a character is typed in the input
  field. This results in a lot of operations on each keypress.
- No need to set this.nodeProperties.nodeBetriebspunktName nor
  this.nodeProperties.nodeBetriebspunktFullName on keyup: this is
  already done automatically by [ngModel].

Note that we do loose a bit of functionality with this patch: the
diagrams and sidebar headings aren't updated in real-time as the
user types anymore.
emersion and others added 10 commits August 19, 2024 17:54
- Times along edges in the editor main view
- Input fields in the train run section form
- Times and input fields in the perlenkette
- Edges in the streckengrafik
- Enforce source/target section ordering
The dark theme doesn't look great when embedded inside OSRD, which
has only a light theme.
The trainrun return is still displayed there. TODO: fix that.
@Uriel-Sautron Uriel-Sautron changed the title Enable edit Enable edit labels Sep 18, 2024
@emersion emersion force-pushed the standalone branch 2 times, most recently from 968e882 to 04c7674 Compare September 18, 2024 16:15
@louisgreiner
Copy link
Collaborator

Label Operation has to be created and pushed to the main repo of NGE instead of our fork

-> This PR has to be closed and another one to be opened to enable only the trainrun labels update component

@clarani clarani deleted the usn/enable-edit-sidebar branch September 19, 2024 14:27
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