Skip to content

Dark theme fixes #1451

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

Merged
nickvergessen merged 7 commits into from
Jan 17, 2019
Merged

Dark theme fixes #1451

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
10cb92d
Fix Chat sidebar in Files for Dark theme
jancborchardt Jan 17, 2019
aa2bfb9
Dark theme: Fix border color in app navigation
jancborchardt Jan 17, 2019
2bade2e
Dark theme: Fix tab header text color
jancborchardt Jan 17, 2019
f36c34d
Remove non-standard background from counter, also emphasizes personal…
jancborchardt Jan 17, 2019
95e69a3
Invert the non-avatar icons so they are visible on dark theme mode
nickvergessen Jan 17, 2019
15a37bd
Fix duplicated double quote
nickvergessen Jan 17, 2019
54b6e88
Add the color for now, so the regex can replace it with white for the…
nickvergessen Jan 17, 2019
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
Invert the non-avatar icons so they are visible on dark theme mode 
Signed-off-by: Joas Schilling <[email protected]>
  • Loading branch information
@nickvergessen
nickvergessen committed Jan 17, 2019
commit 95e69a322e1317ae5fc19312ede7b003f4dfd9e7
17 changes: 12 additions & 5 deletions css/style.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -155,9 +155,7 @@ input[type="password"] {
}

.participantWithList .avatar,
#app-navigation .avatar,
#app-navigation .icon-contacts-dark,
#app-navigation .app-navigation-entry-link .icon-public {
#app-navigation .app-navigation-entry-link > .avatar {
position: absolute;
left: 6px;
top: 6px;
Expand All @@ -170,8 +168,8 @@ input[type="password"] {
margin: 0 !important;
}

.select2-result .icon-contacts.avatar,
.select2-result .icon-public-white.avatar,
.select2-result .icon-contacts-dark.avatar,
.select2-result .icon-public.avatar,
.select2-result .icon-mail.avatar,
#app-navigation .app-navigation-entry-link .avatar.icon {
border-radius: 50%;
Expand Down Expand Up @@ -1070,3 +1068,12 @@ input[type="password"] {
#app-sidebar #participantsTabView .participant:last-child {
margin-bottom: 15px;
}

/**
* Dark-Theme fixes
*/
#app-navigation > ul > li > a:first-child {
img {
filter: none !important;
}
}
4 changes: 2 additions & 2 deletions js/app.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,11 +153,11 @@
},
formatResult: function (element) {
if (element.type === "createPublicRoom") {
return '<span><div class="avatar icon-public-white"></div>' + escapeHTML(element.displayName) + '</span>';
return '<span><div class="avatar icon-public"></div>' + escapeHTML(element.displayName) + '</span>';
}

if (element.type === "createGroupRoom" || element.type === 'group') {
return '<span><div class="avatar icon-contacts"></div>' + escapeHTML(element.displayName) + '</span>';
return '<span><div class="avatar icon-contacts-dark"></div>' + escapeHTML(element.displayName) + '</span>';
}

return '<span><div class="avatar" data-user="' + escapeHTML(element.id) + '" data-user-display-name="' + escapeHTML(element.displayName) + '"></div>' + escapeHTML(element.displayName) + '</span>';
Expand Down
2 changes: 1 addition & 1 deletion js/views/participantview.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@
if (element.type === 'emails') {
return '<span><div class="avatar icon-mail"></div>' + escapeHTML(element.displayName) + '</span>';
} else if (element.type === 'groups') {
return '<span><div class="avatar icon-contacts"></div>' + escapeHTML(element.displayName) + '</span>';
return '<span><div class="avatar icon-contacts-dark"></div>' + escapeHTML(element.displayName) + '</span>';
}

return '<span><div class="avatar" data-user="' + escapeHTML(element.id) + '" data-user-display-name="' + escapeHTML(element.displayName) + '"></div>' + escapeHTML(element.displayName) + '</span>';
Expand Down
8 changes: 4 additions & 4 deletions js/views/roomlistview.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,13 +156,13 @@
templateContext: function() {
var icon = '';
if (this.model.get('objectType') === 'file') {
icon = 'icon icon-file-white';
icon = 'icon icon-file';
} else if (this.model.get('objectType') === 'share:password') {
icon = 'icon icon-password-white';
icon = 'icon icon-password';
} else if (this.model.get('type') === OCA.SpreedMe.app.ROOM_TYPE_GROUP) {
icon = 'icon icon-contacts';
icon = 'icon icon-contacts-dark';
} else if (this.model.get('type') === OCA.SpreedMe.app.ROOM_TYPE_PUBLIC) {
icon = 'icon icon-public-white';
icon = 'icon icon-public';
}

// If a room is a one2one room it can not be removed from the list, only be deleted for both participants.
Expand Down