As long as it's not, as Olin said, "dumping the layout on us", it should be fine. The example you gave isn't that bad because it's at least asking about a specific thing about their layout, although a little more specific would certainly be nice. A bad question would be "Does anything look wrong here?". Keep it specific enough that the question includes few, if any "how about now?" updates. Any such updates must be very clearly marked and the original layout should be kept in the question for future reference.
We're not really big enough to support a second Design Review site, maybe a design-review tag would be a good idea if there isn't one already. Edit: Apparently meta tags like design-review are explicitly discouraged due to their subjective meaning. That being the case, nudging the question towards specific topics is even more important.
In the interest of having design review questions be as specific and answerable as possible, I offer some guidelines for asking a good PCB design review question. Some of these are good practice in general, but are even more important when trying to ask a useful design review question.
Help us help you (what are we looking at?)
- Show the layout in the most useful format your project allows (don't get fired). Design files or Gerbers are best if possible. Layout screenshots should be fairly high resolution and annotated enough to make up for not being design files.
- Include a schematic of the relevant portions of the board. Label nets in the layout accordingly. Nobody should have to guess at what kind of traces they're looking at.
- All components should have reference designators. Use them in your question.
- What are the relevant capabilities of your fabrication/assembly companies? What soldering method is being used?
- Include a size reference of some sort in the layout if necessary (usually a grid).
Narrowing question scope
- If your question is about a specific part of the layout, highlight/explain where it is or crop screenshots accordingly (depending on how the layout is presented in the question).
- Highlight specific components/traces/pours that you are concerned about.
Examples of topics to ask about
This is not an exhaustive list. As a general rule, questions should have more tags than just a hypothetical design-review tag because you want a second opinion about some specific issue.
- Reference (power/ground) planes
- Multi-layer stackups
- Connection of analog and digital reference planes
- Best practices for vias to power/ground planes
- EMI/EMC/noise immunity and related grounding issues
- Guard rings/traces
- Crosstalk
- Unintentional antenna loops
- Sensitive signals (analog or high-speed digital)
- Transmission lines
- Ground return paths
- Component placement
- Component spacing for manufacturability
- Decoupling capacitor placement
- Thermals/heatsinking
- Hotspots
- Thermal reliefs
- Connection of IC thermal pads to copper heatsinking pours
- Trace width