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

new web ui for working with cars #13115

Merged
merged 22 commits into from
May 24, 2024
Merged

new web ui for working with cars #13115

merged 22 commits into from
May 24, 2024

Conversation

mstevetodd
Copy link
Contributor

supports editing location and status
has sort and filter
new link from Conductor shows cars for current train
not (yet) internationalized or snippetized

Copy link

mergeable bot commented May 23, 2024

Thanks for the PR. Please consider adding a release note in the help/en/releasenotes/current-draft-note.shtml file.

@mergeable mergeable bot added the Needs L10N Needs localization label May 23, 2024
Copy link

mergeable bot commented May 23, 2024

Thanks for the PR. It includes changes to properties files, so the 'Needs L10N' label has been added"

@mergeable mergeable bot added the New PR Newly created PR that is less than 1 day old label May 23, 2024
Comment on lines +6 to +23
<meta name="description" content="">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<link rel="stylesheet" href="/css/bootstrap.min.css">
<link rel="stylesheet" href="/css/bootstrap-theme.min.css">
<link rel="stylesheet" href="/css/main.css">
<link rel="stylesheet" href="/css/cars.css">
<link rel="stylesheet" href="/css/user.css">
<script src="/js/jquery-2.2.4.min.js"></script>
<script src="/js/bootstrap.min.js"></script>
<script src="/js/modernizr-2.6.2-respond-1.1.0.min.js"></script>
<script src="/js/json2.js"></script>
<script src="/js/jquery.websocket.js"></script>
<script src="/js/logger.js"></script>
<script src="/js/jquery.jmri.js"></script>
<script src="/js/main.js"></script>
<script src="/js/sorttable.js"></script>
<script src="/js/cars.js"></script>
<script src="/js/user.js"></script>
Copy link
Contributor

Choose a reason for hiding this comment

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

No need to change, but these lines are not indented.

@danielb987 danielb987 removed the New PR Newly created PR that is less than 1 day old label May 24, 2024
@danielb987 danielb987 merged commit a02c286 into JMRI:master May 24, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs L10N Needs localization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants