Commit 6652f3b8 authored by David Haynes's avatar David Haynes 🙆
Browse files

Merge branch 'requiredmsg' into 'master'

Move "Required" under the "Long URL" text to be consistant with Step 2

![Screenshot of modified form](http://i.imgur.com/uzLSHkA.png)

"Required" in its previous position seemed a little confusing and inconsistent. Opened as a MR since this is a matter of opinion. 

See merge request !19
parents ab9f6a09 98e058b5
...@@ -11,7 +11,7 @@ class URLForm(forms.ModelForm): ...@@ -11,7 +11,7 @@ class URLForm(forms.ModelForm):
# Custom target URL field # Custom target URL field
target = forms.URLField( target = forms.URLField(
required=True, required=True,
label='Long URL', label='Long URL (Required)',
max_length=1000, max_length=1000,
widget=forms.URLInput(attrs={ widget=forms.URLInput(attrs={
}) })
...@@ -77,7 +77,7 @@ class URLForm(forms.ModelForm): ...@@ -77,7 +77,7 @@ class URLForm(forms.ModelForm):
<br /> <br />
"""), """),
PrependedText('target', PrependedText('target',
'https:// required', 'https://',
), ),
style="background: rgb(#F6F6F6);", style="background: rgb(#F6F6F6);",
title="target_url", title="target_url",
...@@ -94,7 +94,7 @@ class URLForm(forms.ModelForm): ...@@ -94,7 +94,7 @@ class URLForm(forms.ModelForm):
<br /> <br />
"""), """),
PrependedText('short', PrependedText('short',
'go.gmu.edu/', 'https://go.gmu.edu/',
), ),
style="background: rgb(#F6F6F6);", style="background: rgb(#F6F6F6);",
title="short_url", title="short_url",
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment