documentation icon indicating copy to clipboard operation
documentation copied to clipboard

Add Screenshots?

Open 3dogMcNeill opened this issue 10 years ago • 11 comments

Just went through the exercise on https://5.docs.plone.org/working-with-content/managing-content/ploneformgen/sql_crud

Under "Inserting Rows" where it has "Connection ID" and "Arguments" (twice), can we add a screenshot? I'm too embarassed to say how much time I just wasted because I misunderstood this line: "On separate lines, specify "string1" and "string2". (Leave off the quotes.)"

A picture is worth a thousand words. I'm happy to add the screenshot but I think they're automatically generated somehow?

3dogMcNeill avatar Dec 09 '15 16:12 3dogMcNeill

Since I haven't quite mastered the art of "adding a few extra products to the testing fixture in order to do screenshots", having a manual screenshot is much better than nothing, so please go ahead. There's quite a few in this folder already.

Important:

  • use PNG as the file format if possible
  • filenames without spaces, special chars and such. sql_crud.png would be optimal

polyester avatar Dec 09 '15 16:12 polyester

Great! I will do that. Thanks!

On Wed, Dec 9, 2015 at 11:54 AM, polyester [email protected] wrote:

Since I haven't quite mastered the art of "adding a few extra products to the testing fixture in order to do screenshots", having a manual screenshot is much better than nothing, so please go ahead. There's quite a few in this folder already.

Important:

  • use PNG as the file format if possible
  • filenames without spaces, special chars and such. sql_crud.png would be optimal

— Reply to this email directly or view it on GitHub https://github.com/plone/documentation/issues/466#issuecomment-163322377 .

Christina McNeill IRC: cmcneill

3dogMcNeill avatar Dec 09 '15 21:12 3dogMcNeill

Just went to try to do this....how the heck do I add an image anyway? I'm editing through the GitHub web interface. Maybe I should clone the repo and do it locally, but in what folder should I put the images?

On Wed, Dec 9, 2015 at 4:28 PM, Christina McNeill [email protected] wrote:

Great! I will do that. Thanks!

On Wed, Dec 9, 2015 at 11:54 AM, polyester [email protected] wrote:

Since I haven't quite mastered the art of "adding a few extra products to the testing fixture in order to do screenshots", having a manual screenshot is much better than nothing, so please go ahead. There's quite a few in this folder already.

Important:

  • use PNG as the file format if possible
  • filenames without spaces, special chars and such. sql_crud.png would be optimal

— Reply to this email directly or view it on GitHub https://github.com/plone/documentation/issues/466#issuecomment-163322377 .

Christina McNeill IRC: cmcneill

Christina McNeill IRC: cmcneill

3dogMcNeill avatar Dec 18 '15 21:12 3dogMcNeill

I'm not 100% on how to do it through the GitHub web interface. When cloning locally, it's a simple "git add mypicture.png".

As for folders: just put in the same one as the .rst file is in, that's the easiest to keep track of.

polyester avatar Dec 18 '15 22:12 polyester

ah, there is that suspicious 'new file' button on the webinterface; so through the web should be:

  • create branch
  • hit the "new file" button on the correct directory
  • and this would be a good test of how easy the GitHub webinterface is from there ;-)

polyester avatar Dec 18 '15 22:12 polyester

Ok, I have literally spent 4 hours trying to add 4 screenshots and make some clarification edits. I cloned the repo locally, I tried using GitHub Desktop—couldn't figure out how to make commits to 4.3 branch through command line; with Desktop, ended up making PR to master when I thought I was pushing to "my" local version—it didn't work, I can't find the files I added or the ones I changed, so I quit.

This is way too hard! And I REALLY want to help make the documentation better! Here's the thing: I'm not a programmer, I never will be, and I refuse to become one in order to make writing changes.

On Fri, Dec 18, 2015 at 5:18 PM, polyester [email protected] wrote:

ah, there is that suspicious 'new file' button on the webinterface; so through the web should be:

  • create branch
  • hit the "new file" button on the correct directory
  • and this would be a good test of how easy the GitHub webinterface is from there ;-)

— Reply to this email directly or view it on GitHub https://github.com/plone/documentation/issues/466#issuecomment-165909924 .

Christina McNeill IRC: cmcneill

3dogMcNeill avatar Dec 21 '15 16:12 3dogMcNeill

And also, screenshots can't be added through the web.

On Mon, Dec 21, 2015 at 11:58 AM, Christina McNeill [email protected] wrote:

Ok, I have literally spent 4 hours trying to add 4 screenshots and make some clarification edits. I cloned the repo locally, I tried using GitHub Desktop—couldn't figure out how to make commits to 4.3 branch through command line; with Desktop, ended up making PR to master when I thought I was pushing to "my" local version—it didn't work, I can't find the files I added and changed so I quit.

This is way too hard! And I REALLY want to help make the documentation better! Here's the thing: I'm not a programmer, I never will be, and I refuse to become one in order to make writing changes.

On Fri, Dec 18, 2015 at 5:18 PM, polyester [email protected] wrote:

ah, there is that suspicious 'new file' button on the webinterface; so through the web should be:

  • create branch
  • hit the "new file" button on the correct directory
  • and this would be a good test of how easy the GitHub webinterface is from there ;-)

— Reply to this email directly or view it on GitHub https://github.com/plone/documentation/issues/466#issuecomment-165909924 .

Christina McNeill IRC: cmcneill

Christina McNeill IRC: cmcneill

3dogMcNeill avatar Dec 21 '15 16:12 3dogMcNeill

I'm sorry; that is something we will have to improve then. (also for safety; you deleted the whole 4.3 branch in the process. Easily enough to restore, but that means the process is not well enough documented)

If you could do the following, I will try to make it into a much better non-programmer guideline:

  • send me (old-school, by email) the updated .rst files, and the screenshots)

again, sorry for the timeloss, but let's see if we can turn this around positively by creating a clearer step-by-step guide, using these edits as example.

polyester avatar Dec 21 '15 17:12 polyester

ah, found them already. https://github.com/plone/documentation/commit/aeaed22b874ace77f3c8bb529ff455a3226bb6d8 and https://github.com/plone/documentation/commit/e3eda04ab10eab3d881c1e9e661adb37ea41816b, so to see.

I'll work from there

polyester avatar Dec 21 '15 17:12 polyester

omg, I did?! yikes. sorry.

After I made the PR, I wasn't sure how to stop it from being added so I closed it. I thought that was closing my PR, not deleting the whole branch.

3dogMcNeill avatar Dec 21 '15 17:12 3dogMcNeill

no problem, it shows we need better documentation ;-)

I'll have to find a computer where I can install github-desktop, then make some screenshots using that.

polyester avatar Dec 21 '15 18:12 polyester