Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Fixes #476 Printing a patient record #525

Merged
merged 21 commits into from
Jul 11, 2016
Merged
Show file tree
Hide file tree
Changes from 6 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion app/locales/de/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,8 @@ export default {
no_users_found: 'Keine Benutzer gefunden.',
are_you_sure_delete: 'Wollen Sie den Benutzer {{user}} sicher löschen?',
user_has_been_saved: 'Der Benutzer wurde gespeichert.',
user_saved: 'Benutzer gespeichert'
user_saved: 'Benutzer gespeichert',
for_authorized_persons: 'Dieser Bericht ist nur für autorisierte Personen.'
},
buttons: {
complete: 'Abschließen',
Expand Down
3 changes: 2 additions & 1 deletion app/locales/en/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -312,7 +312,8 @@ export default {
on_behalf_of: 'on behalf of',
new_patient_has_to_be_created: 'A new patient needs to be created...Please wait..',
no_notes_available: 'No additional clinical notes are available for this visit.',
sorry: 'Sorry, something went wrong...'
sorry: 'Sorry, something went wrong...',
for_authorized_persons: 'This report is for authorized persons only.'
},
alerts: {
please_wait: 'Please Wait'
Expand Down
3 changes: 2 additions & 1 deletion app/locales/es-CO/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,8 @@ export default {
on_behalf_of: 'a nombre de',
new_patient_has_to_be_created: 'Un nuevo paciente debe ser creado...Por favor espere..',
no_notes_available: 'No hay notas clínicas adicionales están disponibles para esta visita.',
sorry: 'Lo sentimos, un error ha ocurrido...'
sorry: 'Lo sentimos, un error ha ocurrido...',
for_authorized_persons: 'Este informe es sólo para personas autorizadas.'
},
alerts: {
please_wait: 'Por favor espere'
Expand Down
3 changes: 2 additions & 1 deletion app/locales/es/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,8 @@ export default {
on_behalf_of: 'a nombre de',
new_patient_has_to_be_created: 'Un nuevo paciente debe ser creado...Por favor espere..',
no_notes_available: 'No hay notas clínicas adicionales están disponibles para esta visita.',
sorry: 'Lo sentimos, un error ha ocurrido...'
sorry: 'Lo sentimos, un error ha ocurrido...',
for_authorized_persons: 'Este informe es sólo para personas autorizadas.'
},
alerts: {
please_wait: 'Por favor espere'
Expand Down
3 changes: 2 additions & 1 deletion app/locales/fr/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,8 @@ export default {
on_behalf_of: 'de la part de',
new_patient_has_to_be_created: 'Un nouveau patient doit être crée...Veuillez attendre...',
no_notes_available: 'Aucune note clinique supplémentaire disponible pour cette visite.',
sorry: 'Désolé, quelque chose a mal tourné ...'
sorry: 'Désolé, quelque chose a mal tourné ...',
for_authorized_persons: 'Ce rapport est destiné uniquement aux personnes autorisées.'
},
alerts: {
please_wait: 'Veuillez attendre'
Expand Down
3 changes: 2 additions & 1 deletion app/locales/pt-BR/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ export default {
},
messages: {
no_items_found: 'Nenhum item encontrado',
create_new_record: 'Criar uma nova gravação?'
create_new_record: 'Criar uma nova gravação?',
for_authorized_persons: 'Este relatório é apenas para pessoas autorizadas.'
},
buttons: {
complete: 'Completo',
Expand Down
3 changes: 2 additions & 1 deletion app/locales/ru/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ export default {
},
messages: {
no_items_found: 'Ничего не найдено.',
create_new_record: 'Создать новую запись?'
create_new_record: 'Создать новую запись?',
for_authorized_persons: 'Этот отчет предназначен только для уполномоченных лиц.'
},
buttons: {
complete: 'Готово',
Expand Down
3 changes: 2 additions & 1 deletion app/locales/tr/translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ export default {
},
messages: {
no_items_found: 'Hiçbir öğe bulunamadı.',
create_new_record: 'Yeni bir kayıt oluştur?'
create_new_record: 'Yeni bir kayıt oluştur?',
for_authorized_persons: 'Bu rapor yetkili kişiler içindir.'
},
buttons: {
complete: 'Bitir',
Expand Down
19 changes: 19 additions & 0 deletions app/styles/_print.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
.print-section { display: none; }

@media print {

body {
background: white !important;
color: black !important;
font-size: 12pt;
Copy link
Member

Choose a reason for hiding this comment

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

Should use px values here.

margin: 0 5%;
Copy link
Member

Choose a reason for hiding this comment

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

Selector order is wrong in this block.

&.ember-application { padding-left: 0; }
}

.print-section { display: block; text-align: center; }
Copy link
Member

Choose a reason for hiding this comment

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

Same here: those declarations should each be on a new line.


.panel-footer, .btn, .nav, .view-current-title { display: none; }
Copy link
Member

Choose a reason for hiding this comment

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

Each selector should be on their own line. In fact I thought we had a test for that in the Stylelint. Wonder why it's not failing the build? cc @billybonks


.patient-history-heading { border-bottom: 1px solid black; }
Copy link
Member

Choose a reason for hiding this comment

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

Should only be using a variable, not a color keyword.


}
8 changes: 0 additions & 8 deletions app/styles/_temp_misc.scss
Original file line number Diff line number Diff line change
Expand Up @@ -99,11 +99,3 @@ $table-header-color: rgba($navy_drk2,.7);
from { transform: scale(1) rotate(0deg); }
to { transform: scale(1) rotate(360deg); }
}

@media print {
body { font-size: 10px; }

.ember-application { padding-left: 0; }

.panel-footer { display: none; }
}
1 change: 1 addition & 0 deletions app/styles/app.scss
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
@import 'base';
@import 'layout';
@import 'index';
@import 'print';
@import 'temp_misc';
@import 'sign_in_screen';
@import 'loading_notice';
Expand Down
1 change: 1 addition & 0 deletions app/templates/section.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

<div class="view-top-bar">
<h1 class="view-current-title">{{currentScreenTitle}}</h1>
<h3 class="print-section">{{t 'messages.for_authorized_persons'}}</h3>
<div class="view-action-buttons">
{{#if newButtonAction}}
<button class="btn btn-primary" {{action newButtonAction}}>{{newButtonText}}</button>
Expand Down