Skip to content

TT-664 fix bad UX #872

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.

Sign up for GitHub

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

Closed
wants to merge 8 commits into from
Closed

TT-664 fix bad UX #872

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
dd90c81
TT-664 fix bad UX
ngalvare10 May 19, 2022
1874f43
TT-664 fix bad UX console log
ngalvare10 May 19, 2022
5a19ec4
TT-664 fix bad UX review
ngalvare10 May 19, 2022
4b9c606
TT-664 fix bad UX review
ngalvare10 May 19, 2022
ce73ef8
TT-664 fix bad UX review
ngalvare10 May 19, 2022
30e3d22
TT-664 fix bad UX review
ngalvare10 May 24, 2022
6550335
TT-664 fix bad UX QA
ngalvare10 May 26, 2022
4fe8fd8
TT-664 fix bad UX QA
ngalvare10 May 26, 2022
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
Prev Previous commit
Next Next commit
TT-664 fix bad UX review
  • Loading branch information
@ngalvare10
ngalvare10 committed May 24, 2022
commit 30e3d2277ceee68e5708f87692bd4480ff379b30
4 changes: 2 additions & 2 deletions src/app/modules/reports/components/time-entries-table/time-entries-table.component.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,11 @@ export class TimeEntriesTableComponent implements OnInit, OnDestroy, AfterViewIn
this.rerenderTableSubscription = this.reportDataSource$.subscribe((ds) => {
this.sumDates(ds.data);
});
this.uploadUsers();
this.uploadUsers();
}

ngAfterViewInit(): void {
this.rerenderDataTable();
this.rerenderDataTable();
}

ngOnDestroy(): void {
Expand Down
6 changes: 3 additions & 3 deletions src/app/modules/time-entries/pages/time-entries.component.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@
Add new entry
Copy link
Contributor

@sbateca sbateca May 24, 2022

Choose a reason for hiding this comment

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

Review the whitespaces and the indent

</button>
<button type="button" (click)="onDisplayModeChange()" class="btn btn-primary float-right" *ngIf="isFeatureToggleCalendarActive">
<em class="fas fa-list" *ngIf="displayGridView"></em>
<em class="fas fa-th" *ngIf="!displayGridView"></em>
</button>
<em class="fas fa-list" *ngIf="displayGridView"></em>
<em class="fas fa-th" *ngIf="!displayGridView"></em>
</button>
</div>
<div style="height: 15px"></div>
<app-month-picker [selectedDate]="selectedDate" (dateSelected)="dateSelected($event)"></app-month-picker>
Expand Down