Closed Samantha-Mcguigan closed 6 years ago
Here are a couple of short posts about the problem of drop-down menus for site navigation:
https://www.joedolson.com/2013/07/designing-accessible-navigation/
The second article discusses accessible design features, and while it offers some recommended ways to code pages, the real takeaway here is awareness of what can go wrong, and designing so that the CSS and JavaScript aren't essential to navigate around the site if these scripts ever don't work.
(I'm redesigning some of my own sites to get rid of my old drop-down menus--so all of this is in-process for me, too.)
@jonhoranic @ajnewton1 Are one of you working on the site right now?
@Samantha-Mcguigan I will not be working on anything until tomorrow after my morning final, as I am under a severe time crunch finishing these papers for turn in (tonight and tomorrow).
@jonhoranic I was just trying to work on some things and the header was changed and it wasn't me who did it.
@Samantha-Mcguigan I noticed that there were some changes made, @ajnewton1 has done some of that work so far but I am unsure if he has fully finished that yet. All that would require is copy/paste work for pages that do not have it yet. If there are problems occurring because of it note what they are here so we do not overwrite each others edits.
@Samantha-Mcguigan @ajnewton1 @jonhoranic Just a heads up: Something just recently changed rather drastically in your site's CSS.
I'd encourage you to test changes locally before rewriting each other's work on the website. It looks like the last push to the repo was mine from two days ago, so presumably you can retrieve the CSS as it was. (You can always retrieve it from the commit history of the repo on the web, too.) Sam, you mentioned the problem of hyperlinks coming out in white on white background. That was set for the links on the nav bar, but you want to be putting a class or something on the links that appear in the body of the document.
Alternatively, you could rewrite the CSS so that links in the navbar have a special class="nav", and style those only to be white, and then style the other links differently. But please announce and coordinate with each other on this.
@ajnewton1 @jonhoranic @ebeshero I added to the CSS to create clickable links on the source pages (NPR.html) the menu at the top has also been changed but I don't think that was from me
I went back and copied what was in Git Hub to the CSS and uploaded it to the site and the menu bar is still the same without the changes I made so its not from me
I wonder if someone changed the SSI file: menu.html?
Maybe @ajnewton1 is working on it?
I'll check the last time it was edited on the server (we should be able to see that...)
Nope--the menu.html file remains as it was from April 24. I'll take a closer look to compare the file on the web server with the files we have on GitHub.
@ebeshero I copy and pasted the CSS on GitHub on to my local computer and uploaded it to the site and it didn't change the menu which is strange because when we had that same CSS on the site 2 days ago it looked different. I'm not sure what the problem is because the it was fine before I started working a few hours ago..
@Samantha-Mcguigan I'm inspecting the repo and the state of the web files--I bet there's an issue with Aaron's working over in his new branch of the repo--I think the master branch is missing the files as he worked on them. I'm taking a look over in his branch and comparing with the code we see on the web server right now.
@Samantha-Mcguigan OHHH! I think I know what happened. It's not your fault (don't worry!)--I'll confirm in a moment. I think you started working with the CSS in your master branch, but Aaron has some special styling over on his branch, which he didn't merge with the master branch.
That means, your CSS file from the master branch is missing the CSS that controls the buttons on the newly styled header. Look at Aaron's CSS here: https://github.com/Samantha-Mcguigan/newsAnalysis/blob/ajn54/Website/style.css
@ajnewton1 and @Samantha-Mcguigan We need to merge your changes to the CSS together on the master branch. I think I can do this by issuing a pull request and asking GitHub to help with the merge... (Presumably Aaron's in a final or something...! But I think we can fix this...) I just taught Aaron how to work in his own branch at your project meeting on Monday, and I think he just forgot to do the pull request (or figured no one else was going to make changes to the CSS, etc).
@ebeshero I'm willing to copy and paste what is on Aaron's CSS into the CSS I have on my computer and just add my changes onto that. Its only a few styling things for the links if that would be easier.
@Samantha-Mcguigan That's effectively doing the merge by hand, and we may need to do that anyway, but let me take a look and compare the branch content first. I think we liked Aaron's changes, right? So maybe we should just commit them to the master branch anyway so we don't risk overwriting the website content again on other files!
@ebeshero Sounds good to me! I will continue working on adding the links in my local branch in the meantime. By the way @jonhoranic while I'm doing this I'm able to see where we are missing hyperlinks in the XML documents so I will fix that while I'm there so you don't have to worry about it.
I was working on the website a few days ago. I think we should meet sometime tomorrow to get everything figured out
@ajnewton1 @Samantha-Mcguigan I think I've resolved the conflict and made it safe to edit the web files again... Everyone do a Git Pull!
@ajnewton1 Make sure you're standing on the master branch when you pull.
Did someone mess with the website?
@ajnewton1 We're fixing it!
@ajnewton1 There was an old copy of the CSS on the master branch... the others didn't have access to your CSS, and made a new edit to style links in the bodies of the HTML file. I'm about to replace what's on the web with your CSS. @Samantha-Mcguigan Check the style.css that you just pulled in from the "master" branch--I don't think you committed your changes to the master branch, did you?
@Samantha-Mcguigan @ajnewton1 I'm going to put the CSS that I just pulled from Aaron's branch back up on the website. Sam, I think it's going to need your edits, still, so your links will probably go back to being white again.
And go take a look at the website again! (whew!) Menu restored. :-)
@Samantha-Mcguigan Add your edits to this revised CSS currently on your master branch of GitHub, and then push them back.
@ajnewton1 As you're working in your own separate "ajn" branch, you just need to send pull requests over the master branch so the others have access to your files. It's important so someone doesn't accidentally overwrite your updates to the website.
Rule of thumb for all of you: once it's worthy to be posted on the website, it MUST be committed to the master branch of your project repo, so the entire team is using up-to-date files.
@ebeshero I just pulled and I'm getting an error that my local changes will be overwritten by merge. Should I push my changes first?
@Samantha-Mcguigan That's okay, and actually good because it means GitHub is recognizing your changes. Go ahead and commit your changes first. Then do git status
and report what you see.
Your branch is behind 'origin/master' by 5 commits, and can be fast-forwarded. (use "git pull" to update your local branch)
@Samantha-Mcguigan OK--good! Go ahead and git pull
.
@Samantha-Mcguigan everything okay? Or merge conflict?
It saying the files will be overwritten by merge
Its telling me to commit or stash them
Hmmm. Did you make lots of changes you won't remember how to do again, or do you think it's best to start over with fresh files?
No I can start over its no big deal
OK...let's give git stash
a try...
...if nothing happens, type git status
.
Saved working directory and index state WIP on master: 6a17908 making spot repair to menu.html so it fits into the whole as well-formed HTML HEAD is now at 6a17908 making spot repair to menu.html so it fits into the whole as well-formed HTML
YAY!
Do a git pull
now
whoop!
All in?
Yep! Am good to start working again?
@ajnewton1 @Samantha-Mcguigan I'm going to check to make sure the pages are well-formed in HTML: taking a peek in the w3c Validator. PAUSE a moment while I do this.
https://validator.w3.org/nu/?doc=http%3A%2F%2Fnewsanalysis.newtfire.org%2F
Nothing serious, but you guys do have the xml line at the top. It's not a HUGE deal, but @ajnewton1 some of the pages might not be showing the search icon (need the UTF8 signal).
@Samantha-Mcguigan Yes, I think you can go ahead and work in your changes. And push to the master branch when you're ready.
@ajnewton1 I'd maybe be sure you're working in the master branch at this point, or pull in Sam's changes before you begin anything...(or start a new branch after Sam's done updating...)
@ebeshero @jonhoranic @ajnewton1 On some of the pages text and images are being cut off at the bottom by the blue bar and I'm not sure why. Any ideas?
@Samantha-Mcguigan Put a few </br>
right before the blue bar.
[x] @ajnewton1 moving the drop down menus, page titles, and other website navigation. Make the News Analysis title be its own link to "home". Add link to the GitHub repo somewhere!
[x] @ajnewton1 add the search feature (with code) so it's active on the menu bar.
[x] @jonhoranic search.php: add
<div>
elements and other features for display of pages.[x] @jonhoranic spelling/grammar(!), xml editing (look for files missing
<hyperlink>
elements and look for weird tagging issues[x] @Samantha-Mcguigan finish write ups for network graphs, fix how SVG elements are sitting on the website, adding photos to the about the team page, add links to articles and new header to source pages
[x] (you should probably do this together): Reflection and conclusion on the WHOLE project (what major things did you learn?) Feature this on the home page of the website.