History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: UP-1931
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Jen Bourey
Reporter: Tuyhang Ly
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
uPortal

User Preferences option bullets

Created: 23/Jan/08 02:22 PM   Updated: 18/Apr/08 10:55 AM
Component/s: User Interface
Affects Version/s: 3.0.0-M5, 3.0.0-RC2
Fix Version/s: 3.0.0-RC3

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown


 Description  « Hide
In User Preferences view, the <img> element for bullet.gif is inside of the <a> element for "Choose a skin" and "Select a Locale" options. They should be outside of the <a> element as it is for "Revert to default layout" option.

<a onmouseover="window.status=''; return true;" href="render.userLayoutRootNode.target.ctf1.uP?action=manageSkins">
  <img title="" src="media/org/jasig/portal/channels/DLMUserPreferences/tab-column/bullet.gif" alt="" border="0" height="9" hspace="7" width="9">
Choose a skin
</a>

<img title="" src="media/org/jasig/portal/channels/DLMUserPreferences/tab-column/bullet.gif" border="0" height="9" hspace="7" width="9">
<a onclick="return confirm('You are about to replace your current layout with a default layout. You cannot undo these changes. Do you want to continue?')" href="render.userLayoutRootNode.target.ctf1.uP?action=resetLayout">
Revert to default layout
</a>

<a onmouseover="window.status=''; return true;" href="render.userLayoutRootNode.target.ctf1.uP?uP_fname=user-locales-selector">
  <img title="" src="media/org/jasig/portal/channels/DLMUserPreferences/tab-column/bullet.gif" alt="" border="0" height="9" hspace="7" width="9">
Select a Locale
</a>

 All   Comments   Work Log   Change History      Sort Order:
Jen Bourey [25/Mar/08 11:05 AM]
It looks like the intention was for all of the options bullets to reside inside the links so that they'd be clickable as well as the link text. I've moved the bullet for the revert layout option into the link, so they're all the same now. If we decide it makes more sense to have the bullet icons outside the links, I can do that, but at least everything is consistent now.