Bladeren bron

Generate legacy image placeholder text by taking first letters

https://github.com/nextcloud/server/issues/19335
Signed-off-by: Sergey Shliakhov <husband.sergey@gmail.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
tags/v19.0.0beta6
Sergey Shliakhov 4 jaren geleden
bovenliggende
commit
09ecf4b4eb
4 gewijzigde bestanden met toevoegingen van 47 en 3 verwijderingen
  1. 1
    1
      core/js/dist/main.js
  2. 1
    1
      core/js/dist/main.js.map
  3. 43
    0
      core/js/tests/specs/jquery.placeholderSpec.js
  4. 2
    1
      core/src/jquery/placeholder.js

+ 1
- 1
core/js/dist/main.js
Diff onderdrukt omdat het te groot bestand
Bestand weergeven


+ 1
- 1
core/js/dist/main.js.map
Diff onderdrukt omdat het te groot bestand
Bestand weergeven


+ 43
- 0
core/js/tests/specs/jquery.placeholderSpec.js Bestand weergeven

@@ -0,0 +1,43 @@
/**
* Copyright (c) 2019 Serhii Shliakhov <shlyakhov.up@gmail.com>
*
* This file is licensed under the Affero General Public License version 3
* or later.
*
* See the COPYING-README file.
*
*/

describe('jquery.placeholder tests', function() {

var $div;

beforeEach(function() {
$('#testArea').append($('<div id="placeholderdiv">'));
$div = $('#placeholderdiv');
});

afterEach(function() {
$div.remove();
});

describe('placeholder text', function() {
it('shows one first letter if one word in a input text', function() {
spyOn($div, 'html');
$div.imageplaceholder('Seed', 'Name')
expect($div.html).toHaveBeenCalledWith('N');
});

it('shows two first letters if two words in a input text', function() {
spyOn($div, 'html');
$div.imageplaceholder('Seed', 'First Second')
expect($div.html).toHaveBeenCalledWith('FS');
});

it('shows two first letters if more then two words in a input text', function() {
spyOn($div, 'html');
$div.imageplaceholder('Seed', 'First Second Middle')
expect($div.html).toHaveBeenCalledWith('FS');
});
});
});

+ 2
- 1
core/src/jquery/placeholder.js Bestand weergeven

@@ -158,7 +158,8 @@ $.fn.imageplaceholder = function(seed, text, size) {
this.css('font-size', (height * 0.55) + 'px')

if (seed !== null && seed.length) {
this.html(text[0].toUpperCase())
var placeholderText = text.split(' ', 2).map((word) => word[0].toUpperCase()).join('')
this.html(placeholderText);
}
}


Laden…
Annuleren
Opslaan