Ensure clean disply of blurb on homepage. Monthly pick admin form limits blurb to 200 chars. Blurb gets vertically truncated when more than 4 lines. (bug 715577)

This commit is contained in:
Vaidik Kapoor 2012-01-06 17:23:41 +05:30 коммит произвёл Chris Van
Родитель 76749b0414
Коммит 162b059f48
5 изменённых файлов: 16 добавлений и 1 удалений

Просмотреть файл

@ -189,12 +189,14 @@ class OAuthConsumerForm(happyforms.ModelForm):
class MonthlyPickForm(happyforms.ModelForm):
image = forms.CharField(required=False)
blurb = forms.CharField(max_length=200,
widget=forms.Textarea(attrs={'cols': 20,
'rows': 2}))
class Meta:
model = MonthlyPick
widgets = {
'addon': forms.TextInput(),
'blurb': forms.Textarea(attrs={'cols': 20, 'rows': 2})
}

Просмотреть файл

@ -1121,6 +1121,14 @@ class TestMonthlyPick(amo.tests.TestCase):
eq_(MonthlyPick.objects.count(), 2)
eq_(MonthlyPick.objects.all()[1].locale, '')
def test_insert_long_blurb(self):
dupe = initial(self.f)
dupe.update(id='', blurb='x' * 201, locale='en-US')
data = formset(initial(self.f), dupe, initial_count=1)
r = self.client.post(self.url, data)
eq_(r.context['form'].errors[1]['blurb'][0],
'Ensure this value has at most 200 characters (it has 201).')
def test_success_update(self):
d = initial(self.f)
d.update(locale='fr')

Просмотреть файл

@ -178,6 +178,9 @@
max-height: 120px;
}
}
.blurb > p {
height: 68px;
}
}
.html-rtl #monthly .thumb {

Просмотреть файл

@ -31,6 +31,7 @@ function initPromos($context) {
});
$('.addons h3', $promos).truncate({dir: 'h'});
$('.addons .desc', $promos).truncate({dir: 'v'});
$('#monthly .blurb > p').truncate({dir: 'v'});
$('.install', $promos).installButton();
});
$('.toplist .name').truncate({showTitle: true});

Просмотреть файл

@ -61,6 +61,7 @@ function initTrunc() {
// Trim the add-on title and description text to fit.
$('.htruncate').truncate({dir: 'h'});
$('.vtruncate').truncate({dir: 'v'});
$('#monthly .blurb > p').truncate({dir: 'v'});
$(window).resize(debounce(function() {
$('.htruncate').truncate({dir: 'h'});
$('.vtruncate').truncate({dir: 'v'});