Skip to content

Commit

Permalink
Show explicit user name instead of avatar in lists. (#4828)
Browse files Browse the repository at this point in the history
  • Loading branch information
arikfr authored Apr 24, 2020
1 parent cfc8215 commit 6ee9b43
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 6 deletions.
2 changes: 1 addition & 1 deletion client/app/pages/alerts/AlertsList.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ class AlertsList extends React.Component {
field: "name",
}
),
Columns.custom((text, item) => item.user.name, { title: "Created By" }),
Columns.custom.sortable(
(text, alert) => (
<div>
Expand All @@ -59,7 +60,6 @@ class AlertsList extends React.Component {
}
),
Columns.timeAgo.sortable({ title: "Last Updated At", field: "updated_at", className: "text-nowrap", width: "1%" }),
Columns.avatar({ field: "user", className: "p-l-0 p-r-0" }, name => `Created by ${name}`),
Columns.dateTime.sortable({ title: "Created At", field: "created_at", className: "text-nowrap", width: "1%" }),
];

Expand Down
2 changes: 1 addition & 1 deletion client/app/pages/dashboards/DashboardList.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ class DashboardList extends React.Component {
width: null,
}
),
Columns.avatar({ field: "user", className: "p-l-0 p-r-0" }, name => `Created by ${name}`),
Columns.custom((text, item) => item.user.name, { title: "Created By" }),
Columns.dateTime.sortable({
title: "Created At",
field: "created_at",
Expand Down
4 changes: 3 additions & 1 deletion client/app/pages/dashboards/components/DashboardHeader.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,9 @@ function DashboardPageTitle({ dashboardOptions }) {
ignoreBlanks
/>
</h3>
<img src={dashboard.user.profile_image_url} className="profile-image" alt={dashboard.user.name} />
<Tooltip title={dashboard.user.name} placement="bottom">
<img src={dashboard.user.profile_image_url} className="profile-image" alt={dashboard.user.name} />
</Tooltip>
</div>
<DashboardTagsControl
tags={dashboard.tags}
Expand Down
5 changes: 2 additions & 3 deletions client/app/pages/queries-list/QueriesList.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -77,12 +77,11 @@ class QueriesList extends React.Component {
width: null,
}
),
Columns.avatar({ field: "user", className: "p-l-0 p-r-0" }, name => `Created by ${name}`),
Columns.custom((text, item) => item.user.name, { title: "Created By" }),
Columns.dateTime.sortable({ title: "Created At", field: "created_at" }),
Columns.duration.sortable({ title: "Runtime", field: "runtime" }),
Columns.dateTime.sortable({ title: "Last Executed At", field: "retrieved_at", orderByField: "executed_at" }),
Columns.custom.sortable((text, item) => <SchedulePhrase schedule={item.schedule} isNew={item.isNew()} />, {
title: "Update Schedule",
title: "Refresh Schedule",
field: "schedule",
}),
];
Expand Down

0 comments on commit 6ee9b43

Please sign in to comment.