User tests: Successful: Unsuccessful:
see: #5628
This is a proposal UI/UX review on mobile for the redirect component.
Labels |
Added:
?
|
@brianteeman i think this is out of scope of this PR This should only fix the resolution
issue.
:) I suppose I asked for that. I will do a separate PR
On 11 Jan 2015 01:27, "zero-24" notifications@github.com wrote:
@brianteeman https://github.com/brianteeman i think this is out of
scope of this PR [image: ] This should only fix the resolution
issue.—
Reply to this email directly or view it on GitHub
#5665 (comment).
Thanks @brianteeman
Category | ⇒ | Administration UI/UX |
Easy | No | ⇒ | Yes |
@test all good thanks
Status | Pending | ⇒ | Ready to Commit |
thanks @waader
If the scope was to hide several columns on small devices.
Yes the other issue is handeled by @brianteeman with this PR: #5679
Labels |
Added:
?
|
@zero-24 Can you please check your files since they can't be applied. I get the following errors:
error: patch failed: administrator/components/com_redirect/views/links/tmpl/default.php:120
error: administrator/components/com_redirect/views/links/tmpl/default.php: patch does not apply
Your branch may be out of sync.
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-17 09:48:47 |
Labels |
Removed:
?
|
I think there is a bigger UI problem with this component and it effects all
resolutions.
If the plugin is running then it will end up capturing lots of crazy long
URLs that we dont wrap. I would look at the PR that @philetaylor mad to
etap crazy long article titles
On 11 Jan 2015 01:05, "zero-24" notifications@github.com wrote: