diff --git a/www/drive/file.css b/www/drive/file.css index 87d951ccb..489d95050 100644 --- a/www/drive/file.css +++ b/www/drive/file.css @@ -61,9 +61,6 @@ span.fa-folder-open { color: #FEDE8B; text-shadow: -1px 0 #000000, 0 1px #000000, 1px 0 #000000, 0 -1px #000000; } -span.fa { - min-width: 20px; -} /* TREE */ #tree { border-right: 1px solid #ccc; @@ -212,42 +209,44 @@ span.fa { #content div.grid .listElement { display: none; } -#content .list li { - display: flex; - flex-flow: row; - align-items: center; - padding-right: 0px; +#content .list ul { + display: table; + width: 100%; } -#content .list li .element { - display: inline-flex; - flex: 1; +#content .list li { + display: table-row; +} +#content .list li > span { + padding: 0 5px; + display: table-cell; } #content .list li.header { cursor: default; - color: #008; - margin-top: 10px; + color: #888; } -#content .list li.header .element span:not(.fa) { - border-right: 1px solid #CCC; +#content .list li.header span:not(.fa) { text-align: left; } -#content .list li.header .element span.fa { +#content .list li.header span.sortasc, +#content .list li.header span.sortdesc { float: right; } -#content .list li.header span.name { - padding-left: 0; +#content .list li.header > span.active { + font-weight: bold; +} +#content .list li.header > span.clickable { + cursor: pointer; +} +#content .list li.header > span.clickable:hover { + background: #d8d8d8; } #content .list .element span { - padding: 0px 10px; - display: inline-block; overflow: hidden; white-space: nowrap; box-sizing: border-box; - padding-right: 0px; - border-right: 10px solid rgba(0, 0, 0, 0); } -#content .list .element span.name { - width: 478px; +#content .list .element span.icon { + width: 30px; } #content .list .element span.type, #content .list .element span.atime, @@ -256,7 +255,6 @@ span.fa { } #content .list .element span.title { width: 250px; - display: inline; } @media screen and (max-width: 1200px) { #content .list .element span.title { @@ -267,9 +265,6 @@ span.fa { #content .list .element span.files { width: 150px; } -#content .list .header span.name { - width: 500px; -} .parentFolder { cursor: pointer; margin-left: 10px; diff --git a/www/drive/file.less b/www/drive/file.less index f450be3da..dd8de9279 100644 --- a/www/drive/file.less +++ b/www/drive/file.less @@ -70,9 +70,6 @@ span { color: #FEDE8B; text-shadow: -1px 0 black, 0 1px black, 1px 0 black, 0 -1px black; } - &.fa { - min-width: 20px; - } } /* TREE */ @@ -240,55 +237,56 @@ span { } .list { + // Make it act as a table! + ul { + display: table; + width: 100%; + } li { - display: flex; - flex-flow: row; - align-items: center; - padding-right: 0px; - .element { - display: inline-flex; - flex: 1; + display: table-row; + &> span { + padding: 0 5px; + display: table-cell; } + } + li { &.header { cursor: default; - color: #008; - margin-top: 10px; - .element { - span { - &:not(.fa) { - border-right: 1px solid #CCC; - text-align: left; - } - &.fa { - float: right; - } + color: #888; + span { + &:not(.fa) { + text-align: left; + } + &.sortasc, &.sortdesc { + float: right; } } - span { - &.name { - padding-left: 0; + &> span { + &.active { + font-weight: bold; + } + &.clickable { + cursor: pointer; + &:hover { + background: #d8d8d8; + } } } } } .element { span { - padding: 0px 10px; - display: inline-block; overflow: hidden; white-space: nowrap; box-sizing: border-box; - padding-right: 0px; - border-right: 10px solid rgba(0, 0, 0, 0); - &.name { - width: 478px; + &.icon { + width: 30px; } &.type, &.atime, &.ctime { width: 175px; } &.title { width: 250px; - display: inline; @media screen and (max-width: 1200px) { display: none; } @@ -298,13 +296,6 @@ span { } } } - .header { - span { - &.name { - width: 500px; - } - } - } } } diff --git a/www/drive/main.js b/www/drive/main.js index 50c851e44..9f207fb4d 100644 --- a/www/drive/main.js +++ b/www/drive/main.js @@ -183,16 +183,15 @@ define([ // Icons - var $folderIcon = $('', {"class": "fa fa-folder folder"}); + var $folderIcon = $('', {"class": "fa fa-folder folder icon"}); var $folderEmptyIcon = $folderIcon.clone(); - var $folderOpenedIcon = $('', {"class": "fa fa-folder-open folder"); + var $folderOpenedIcon = $('', {"class": "fa fa-folder-open folder"}); var $folderOpenedEmptyIcon = $folderOpenedIcon.clone(); - var $fileIcon = $('', {"class": "fa fa-file-text-o file"}); - var $padIcon = $('', {"class": "fa fa-file-word-o file"}); - var $codeIcon = $('', {"class": "fa fa-file-code-o file"}); - var $slideIcon = $('', {"class": "fa fa-file-powerpoint-o file"}); - var $pollIcon = $('', {"class": "fa fa-calendar file"}); - var $anonIcon = $('', {"class": "fa fa-user-secret file listonly"}); + var $fileIcon = $('', {"class": "fa fa-file-text-o file icon"}); + var $padIcon = $('', {"class": "fa fa-file-word-o file icon"}); + var $codeIcon = $('', {"class": "fa fa-file-code-o file icon"}); + var $slideIcon = $('', {"class": "fa fa-file-powerpoint-o file icon"}); + var $pollIcon = $('', {"class": "fa fa-calendar file icon"}); var $upIcon = $('', {"class": "fa fa-arrow-circle-up"}); var $unsortedIcon = $('', {"class": "fa fa-files-o"}); var $templateIcon = $('', {"class": "fa fa-cubes"}); @@ -202,8 +201,8 @@ define([ var $expandIcon = $('', {"class": "fa fa-plus-square-o expcol"}); var $listIcon = $('', {"class": "fa fa-list"}); var $gridIcon = $('', {"class": "fa fa-th"}); - var $sortAscIcon = $('', {"class": "fa fa-angle-up"}); - var $sortDescIcon = $('', {"class": "fa fa-angle-down"}); + var $sortAscIcon = $('', {"class": "fa fa-angle-up sortasc"}); + var $sortDescIcon = $('', {"class": "fa fa-angle-down sortdesc"}); var $closeIcon = $('', {"class": "fa fa-window-close"}); if (!APP.readOnly) { @@ -235,7 +234,7 @@ define([ $iframe.find('.selected').removeClass("selected"); }; var removeInput = function () { - $iframe.find('li > span:hidden').show(); + $iframe.find('li > span:hidden').attr('style', undefined); $iframe.find('li > input').remove(); }; @@ -367,8 +366,8 @@ define([ return true; } - // $element should be the , find it if it's not the case - var $element = $(e.target).closest('li').children('span.element'); + // $element should be the
  • + var $element = $(e.target).closest('li'); onElementClick(undefined, $element); if (!$element.length) { logError("Unable to locate the .element tag", e.target); @@ -673,22 +672,20 @@ define([ var element = filesOp.findElement(files, newPath); var $icon = !isFolder ? getFileIcon(element) : undefined; - var spanClass = 'file-element element'; - var liClass = 'file-item'; + var liClass = 'file-item file-element element'; if (isFolder) { - spanClass = 'folder-element element'; - liClass = 'folder-item'; + liClass = 'folder-item folder-element element'; $icon = filesOp.isFolderEmpty(root[key]) ? $folderEmptyIcon.clone() : $folderIcon.clone(); } - var $name = $('', { 'class': spanClass }).text(key); - if (isFolder) { - addFolderData(element, key, $name); - } else { - addFileData(element, key, $name, true); - } var $element = $('
  • ', { draggable: true - }).append($icon).append($name).dblclick(function () { + }); + if (isFolder) { + addFolderData(element, key, $element); + } else { + addFileData(element, key, $element, true); + } + $element.prepend($icon).dblclick(function () { if (isFolder) { module.displayDirectory(newPath); return; @@ -946,18 +943,19 @@ define([ $icon = $sortDescIcon.clone(); } if (typeof(Cryptpad.getLSAttribute(SORT_FOLDER_DESC)) !== "undefined") { - $list.find('.foldername').prepend($icon); + $list.find('.foldername').addClass('active').prepend($icon); } }; var getFolderListHeader = function () { - var $folderHeader = $('
  • ', {'class': 'header listElement'}); - var $fohElement = $('', {'class': 'element'}).appendTo($folderHeader); - var $name = $('', {'class': 'name foldername'}).text(Messages.fm_folderName).click(onSortByClick); + var $fohElement = $('
  • ', {'class': 'header listElement'}); + //var $fohElement = $('', {'class': 'element'}).appendTo($folderHeader); + var $fhIcon = $('', {'class': 'icon'}); + var $name = $('', {'class': 'name foldername clickable'}).text(Messages.fm_folderName).click(onSortByClick); var $subfolders = $('', {'class': 'folders listElement'}).text(Messages.fm_numberOfFolders); var $files = $('', {'class': 'files listElement'}).text(Messages.fm_numberOfFiles); - $fohElement.append($name).append($subfolders).append($files); + $fohElement.append($fhIcon).append($name).append($subfolders).append($files); addFolderSortIcon($fohElement); - return $folderHeader; + return $fohElement; }; var addFileSortIcon = function ($list) { var $icon = $sortAscIcon.clone(); @@ -968,19 +966,20 @@ define([ if (Cryptpad.getLSAttribute(SORT_FILE_BY) === '') { classSorted = 'filename'; } else if (Cryptpad.getLSAttribute(SORT_FILE_BY)) { classSorted = Cryptpad.getLSAttribute(SORT_FILE_BY); } if (classSorted) { - $list.find('.' + classSorted).prepend($icon); + $list.find('.' + classSorted).addClass('active').prepend($icon); } }; // _WORKGROUP_ : do not display title, atime and ctime in workgroups since we don't have files data var getFileListHeader = function (displayTitle) { - var $fileHeader = $('
  • ', {'class': 'file-header header listElement'}); - var $fihElement = $('', {'class': 'element'}).appendTo($fileHeader); - var $fhName = $('', {'class': 'name filename'}).text(Messages.fm_fileName).click(onSortByClick); - var $fhTitle = $('', {'class': 'title '}).text(Messages.fm_title).click(onSortByClick); - var $fhType = $('', {'class': 'type'}).text(Messages.table_type).click(onSortByClick); - var $fhAdate = $('', {'class': 'atime'}).text(Messages.fm_lastAccess).click(onSortByClick); - var $fhCdate = $('', {'class': 'ctime'}).text(Messages.fm_creation).click(onSortByClick); - $fihElement.append($fhName); + var $fihElement = $('
  • ', {'class': 'file-header header listElement element'}); + //var $fihElement = $('', {'class': 'element'}).appendTo($fileHeader); + var $fhIcon = $('', {'class': 'icon'}); + var $fhName = $('', {'class': 'name filename clickable'}).text(Messages.fm_fileName).click(onSortByClick); + var $fhTitle = $('', {'class': 'title clickable'}).text(Messages.fm_title).click(onSortByClick); + var $fhType = $('', {'class': 'type clickable'}).text(Messages.table_type).click(onSortByClick); + var $fhAdate = $('', {'class': 'atime clickable'}).text(Messages.fm_lastAccess).click(onSortByClick); + var $fhCdate = $('', {'class': 'ctime clickable'}).text(Messages.fm_creation).click(onSortByClick); + $fihElement.append($fhIcon).append($fhName); if (displayTitle && !isWorkgroup()) { $fihElement.append($fhTitle); } @@ -989,7 +988,8 @@ define([ $fihElement.append($fhAdate).append($fhCdate); } addFileSortIcon($fihElement); - return $fileHeader; + return $fihElement; + //return $fileHeader; }; var allFilesSorted = function () { @@ -1098,11 +1098,12 @@ define([ } var idx = files[rootName].indexOf(href); var $icon = getFileIcon(href); - var $name = $('', { 'class': 'file-element element' }); - addFileData(href, file.title, $name, false); var $element = $('
  • ', { + 'class': 'file-element element', draggable: draggable - }).append($icon).append($name).dblclick(function () { + }); + addFileData(href, file.title, $element, false); + $element.prepend($icon).dblclick(function () { openFile(href); }); var path = [rootName, idx]; @@ -1129,9 +1130,9 @@ define([ var sortedFiles = sortElements(false, [FILES_DATA], keys, Cryptpad.getLSAttribute(SORT_FILE_BY), !getSortFileDesc(), false, true); sortedFiles.forEach(function (file) { var $icon = getFileIcon(file.href); - var $name = $('', { 'class': 'file-element element' }); - addFileData(file.href, file.title, $name, false); - var $element = $('
  • ').append($icon).append($name).dblclick(function () { + var $element = $('
  • ', { 'class': 'file-element element' }) + addFileData(file.href, file.title, $element, false); + $element.prepend($icon).dblclick(function () { openFile(file.href); }); $element.click(function(e) {