Added comments from peoples reviews

FrontendMentor/huddle-landing-page-with-single-introductory-section-master/work-through/myProcess.md

@@ -46,21 +46,30 @@ next day
         its like a pixel off with the social media buttons
     img-diff-2-desktop-near-enough.png
     ![alt](img-diff-2-desktop-near-enough.png)
-
     Still now on to the mobile variant. Can I nail it better?
     spent an age on this, lots of tweaking. Not quite there but nearly, so nearly
+    ![mobileimage](mob-img-diff-1.png)
+    submit to frontend mentor as a solution
 
-    ![alt](mob-img-diff-1.png)
+Couple of days later
-
+    So I had a 3 responses on Frontend Mentor page
-    so. This is where I gave up.
+    1 Comment on Slack
-
+    2 responses on Twitter. 
-    --
+    less than I hoped but some useful information to take on board
-    Assessment: 
+        First: Don't worry about it being so exact(!!), being a little off/up/down/font off doesn't really matter ... ALTHOUGH it CAN pay off to be exact. Theoretically you are hired to implement the required design, so best to be as exact as possible.
-
+        Second: Link color in the attribution is impossible to read. My fault for not making any changes to it.
-    Issues with going in the sizes between 375w and 1440w. Social buttons are no longer right aligned or center aligned.
+        Third: mobile design - make icon size 13x13, that will fix centering
-    Still needs accessibility stuff
+        Fourth: mobile design - the hit box for the link is off from the image/circle
-
+        fifth: look at stacked icons on font awesome https://fontawesome.com/how-to-use/on-the-web/styling/stacking-icons
-Things I've not touched so far:
+        sixth: hover state for social icons not a good color. Aim for a lighter violet. Aim to stay in color scheme (important for accessibility)
-    main section
+        seventh: good not used too much css, nor complex structure
-    accessibility
+        eighth: "personal pet peeve" html button is only valid in an html form, otherwise use a anchor styled as a button because a button has no href. -- Not sure which way to go with this because its not clear what would happen on the button press. Does it go to a new page, or reveal a email submission box
-    mobile
+        ninth: "personally I would have wrapped the social icons and the footer links in a combined foot section just because that makes them easier to move later if doing a whole site". Mixed on this, attribution link at bottom was not part of the original design, but was part of the base files, and is something that was deletable. 
+        tenth: (overall people though I did a good/awesome job)
+        eleventh: was told to check out PerfectPixel extension for Chrome/Firefox. Similar job to diffchecker, can use to add a design image as a layer and you can see how off you are from the design ... being by opacity or there is a difference view
+        twelfth: documenting the process was apparently helpful ("its something not everyone does, but should")
+        thirteenth: Design probably means in wide formats that the page should cover all the screen. Both in height and width. That is more powerful than image should be at X/Y, should be this tall. Text should also be at X/Y and be a certain size, with certain spacing between letters/lines.
+        fourteenth: Limit the width of the components, and contain them to the center of your screen.
+        fifteenth: At present if you go really wide on the page, the text will appear in only one line. It totally loses its expected design. Use a main container to hold everything inside it. Limit its size and automatically center it.
+        sixteenth: I've used flexbox but some of the child elements are not restricted in the size they can take. This si useful to maintain proportions at different sizes. (Good example is the mockup image, which takes up more space than it should in the sizes between roughly 500-800px)
+        seventeenth: Basically ... try not to restrict your designs to two resolutions. Make it work for in-between widths.