Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

menu item widths inconsistent #355

Closed
jnurthen opened this issue Apr 10, 2017 · 6 comments
Closed

menu item widths inconsistent #355

jnurthen opened this issue Apr 10, 2017 · 6 comments
Assignees
Labels
bug Code defects; not for inaccurate prose Example Page Related to a page containing an example implementation of a pattern Feedback Issue raised by or for collecting input from people outside APG task force

Comments

@jnurthen
Copy link
Member

feedback from @carmacleod in #144

Looking at http://w3c.github.io/aria-practices/examples/menubar/menubar-2/menubar-2.html

Checked items in the menu do not have the same width as non-checked items.
They are missing a rectangular piece from the right hand side of the item that seems to have about the same width as the (left-aligned) checkmark.
This makes a dropped-down menu look "staggered", like a vertical stack of 2 x 4 Lego bricks that has some (left-aligned) 2 x 3 Lego bricks in it.

@jongund
Copy link
Contributor

jongund commented Apr 10, 2017

Creating the branch menubar-issue324 and having me pull that into my clone of aria-authoring practices has got me very confused on what the state of my repository is and how I get things back into the w3c repository. Can you clear all my pull requests as soon as possible and I will start again on fixing the remaining menubar issues. I will gut checkout a new copy so I can get reoriented to how to get code back into the w3c repository.

@carmacleod
Copy link
Contributor

@jnurthen I believe the previous comment re "clearing Jon's pull request backlog" was meant for you.

@mcking65
Copy link
Contributor

mcking65 commented Apr 15, 2017

@jongund, sorry that was somehow confusing.

The issue #324 changes are now merged into master, and I have deleted the menubar-issue324 branch.

If you just update your fork and make a new branch from master for your issue 355 changes, you will be building on the latest.

Then, when you submit a PR for issue 355, choose w3c/aria-practices/master as the base branch for the pull request.

Update your fork as described on our
wiki page about pull requests
under the "Synnc Your Fork" heading:

  1. git fetch -p w3c 
  2. git checkout master 
  3. git merge w3c/master 
  4. git push origin 

This assumes the remote for your fork is named "origin" and the remote for the w3c repo is named "w3c". The wiki page describes those assumptions in more detail.

Note: I added the "-p" option to the fetch command to prune off the deleted branches. I will update the wiki accordingly.

@mcking65 mcking65 added bug Code defects; not for inaccurate prose Example Page Related to a page containing an example implementation of a pattern Feedback Issue raised by or for collecting input from people outside APG task force labels Apr 17, 2017
@mcking65 mcking65 added this to the Jan 2017 Clean Up milestone Apr 17, 2017
mcking65 added a commit that referenced this issue Apr 20, 2017
For issue #355, fixed typo in script src attribute. Correct styleManager.js is now loaded and menuitem widths are now consistent.
@mcking65
Copy link
Contributor

@jnurthen, @carmacleod,

With the latest commit, is this issue resolved?

@carmacleod
Copy link
Contributor

Yes, the menus are the correct width now. You can close this issue. Thanks!

@mcking65
Copy link
Contributor

Excellent, thanks @carmacleod! Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Code defects; not for inaccurate prose Example Page Related to a page containing an example implementation of a pattern Feedback Issue raised by or for collecting input from people outside APG task force
Development

No branches or pull requests

4 participants