Comments

lisarex’s picture

The word 'committed' is misspelled as 'comitted' (not assigning this to myself to QA, just making a note :))

shyamala’s picture

UI tested in Chrome and Firefox 3.5

The News, Doc update, Forums, Commits TAB: Doc Updates doesnot have a more link

shyamala’s picture

Assigned: Unassigned » shyamala
Category: task » bug
Status: Active » Needs work

Style QA: Current Activity and This Week on the right side should be Blue in colour -> Eveen if they are not links we should use a more subtle color than black.

shyamala’s picture

Title tags missing for Footer menu, are these generated using Drupal Menu, we need to them add Description.

shyamala’s picture

Important Images like Get Started Now and other captions should have title tag.

shyamala’s picture

1) The Creative Commons Image caption doesnot have a mouse over effect, because the hyperlink for image and caption are grouped. They should be separated.

2) IE 8: The Map doesn't work, jquery error on page
Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0; GTB6.5)
Timestamp: Thu, 7 Oct 2010 19:41:23 UTC

Message: 'jQuery' is undefined
Line: 264
Char: 1
Code: 0
URI: http://beta.drupal.org/

3) IE8: The Ad slot is broken, moves below the page (Drupilcon Chicago Ad)
4) IE8: The Tabs News | Doc Updates | Forum Posts | Commits do not work on IE 8

longwave’s picture

Firefox 3.6: After clicking to change tab, an ugly dotted border is left around the selected tab.

corbacho’s picture

@longwave. It's ugly I know. But it's due of accessibility reasons, so you can navigate through the elements of the webpage with the keyboard. See http://drupal.org/node/428016
Although the prototype has "outline:none" so it doesn't show it. Mmmmh

Everett Zufelt’s picture

There is no reason that a focus indicator needs to be ugly. Focus can be visually indicated in a non-ugly style, the choice of how to visually indicate focus is up to designers. The focus indication must be visually distinct enough that persons with partial vision or color blindness can perceive the difference.

dddave’s picture

StatusFileSize
new25.91 KB

In IE the tooltips/help balloon thingies on the map are missing when some activity is indicated. Only the text is displayed.

The debugger whines about an "unsupported method" and marks this script section:var ctx=canvas.getContext("2d") (line 522 in some cached js)

drumm’s picture

Status: Needs work » Active

I redid the map bubbles, which should fix up the IE errors. Please test out redesign.drupal.org (I haven't updated beta.drupal.org yet).

By-design browser differences which should be okay-
I used CSS to round the corners, so IE 8 and below won't have that.
I used CSS :after for the arrow, so IE 7 and below won't have that.

Would be great to see screenshots of the map in IE 7, 8, and 9.

Everett Zufelt’s picture

If I am testing this component with a screen-reader (I am completely blind) what am I looking for on the page? What is the expected behavior of the component and where does it appear in the page structure?

drumm’s picture

http://redesign.drupal.org/#front-bottom-left-inner should get to it.

It is recent actions by people on Drupal.org, mapped to their location. For example,

webchick comitted
#912218 by Gábor Hojtsy: Add missing…

One map bubble is shown at a time, cycling every 4 seconds. Technically, it is display: none; and jQuery .hide() and .show().

The individual content is a bit random and not too useful. The purpose is to demonstrate that we have an active, worldwide community.

corbacho’s picture

StatusFileSize
new182.23 KB
new181.89 KB

drumm here it goes 2 screenshots. (I don't have installed ie9)

The color dots in the tabs in IE7 are not vertically aligned. (Screenshot from IETester)
And the map bubbles don't have arrows, but probably they are not necessary.

IE8 seems fines.

drumm’s picture

(If you are testing right now, wait about 30 minutes. redesign.drupal.org is rebuilt around midnight PDT every day.)

corbacho’s picture

StatusFileSize
new21.94 KB

I took it a couple of minutes ago again. IE7 . I don't see any difference:

From the screenshot "drumm updated Screenshots and uploading images" Do you drop me a hint? ;)
Cropped png-8 screenshot attached

EDIT. Oops.. it was only 8-colors palette. Next time will be png-8 (256 colors).

Everett Zufelt’s picture

Priority: Normal » Major
Issue tags: +Accessibility

2.2.2 Pause, Stop, Hide: For moving, blinking, scrolling, or auto-updating information, all of the following are true: (Level A)
• Moving, blinking, scrolling: For any moving, blinking or scrolling information that (1) starts automatically, (2) lasts more than five seconds, and (3) is presented in parallel with other content, there is a mechanism for the user to pause, stop, or hide it unless the movement, blinking, or scrolling is part of an activity where it is essential; (http://www.w3.org/TR/WCAG20/)

drumm’s picture

Also from http://www.w3.org/TR/WCAG20/

essential
if removed, would fundamentally change the information or functionality of the content, and information and functionality cannot be achieved in another way that would conform

I would say this is not essential. The same types of content are shown to the right, http://beta.drupal.org/#front-bottom-right-inner, but is not filtered for users who have opted-in to share their approximate location.

I think this is working well enough for me, but will leave it for a QA person to mark fixed.

Everett Zufelt’s picture

The problem with changing content is the following.

1. Does it break functionality? No, this does not, since as @Drumm states, the functionality is duplicated elsewhere.

2. Does it distract some users and make it difficult for them to focus on and understand the other page content? Yes, this type of auto updating information can distract some users and make it difficult for them to focus on other portions of the page.

This * absolutely * needs a fix. It is definitely not critical, as it doesn't break functionality. But, it does require a fix. Of course mine is the opinion of only one accessibility consultant, so you may wish to consult with others.

dstol’s picture

Resolution: 1024x768
Browser: FF 3.6.10, likely all

http://img.skitch.com/20101014-8gthwsu1447q5w5y6kakuwf6b7.jpg

Who 'pdated' what now?

froboy’s picture

I noticed this too. Any item with more than 5 or 6 words which is in the Americas runs off of the side of the window when the window is less than about 1300px wide. I can't tell exactly how the content boxes are being rendered, but would it be possible to somehow set up the styles such that any point with long<-25 (i.e. "on the left of the map") has its content box pushed more to the right?

lisarex’s picture

Priority: Major » Critical
Status: Active » Needs work
Issue tags: +drupal.org redesign qa theme

I've also observed some of the updates getting cut off (see screenshot in comment #20)

lisarex’s picture

Assigned: shyamala » Unassigned
drumm’s picture

Priority: Critical » Normal

Committed some positioning for east and west that will be live in a bit.

Leaving open for an accessible pause button. Everett, any good examples for this? most likely it will be a <a href="javascript:void(0);">pause map</a> link.

drumm’s picture

Category: bug » task
monocat’s picture

Assigned: Unassigned » shyamala
Category: task » bug
Priority: Normal » Major
Status: Needs work » Active
Issue tags: -drupal.org redesign qa theme

Don't want to be rude by not posting any issues, but was wondering what module(s) is used to run this interactive map. Thanks.

lisarex’s picture

Assigned: shyamala » Unassigned
Status: Active » Needs work

Fixing issue settings, unassigning etc.

drumm’s picture

Priority: Major » Normal

Fixing more.

monocat - look for a writeup on Drupal.org in a bit, and please don't reset the issue status.

Everett Zufelt’s picture

@Drumm

I am confused that we call this a "map", what part is "map-like"?

yoroy’s picture

@Everett: the little activity bubbles are shown on an image of a world map

froboy’s picture

JQuery Cycle has some good pause functionality in the form of its "pauseOnHover" and "pauseOnPagerHover" options, the later of which is demonstrated here.

It might make sense to do something like

// please excuse the psudocode
$pins.hover($(this).clearInterval(),$(this).setInterval(...))

in order to pause the rotation when a pin is hovered over and then resume when it is left.

Everett Zufelt’s picture

@froboy

That is likely a good idea, but we will need something more since it will be of no use to keyboard only and screen-reader users. Also not useful to those people who just need the updates to stop because they distract their attention from the rest of the page.

I don't have time to suggest a solution, but one is needed. I am happy to review any concepts others generate.

froboy’s picture

The same clear/set code could be extended to a button such as this or this.

(sorry for the continual jQuery Cycle references, but it's something I know and it's quite well documented and demonstrated)

lisarex’s picture

Status: Needs work » Closed (fixed)

The front page map has been fixed for some time now :)