> On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > Some of the image file names don't match the introductory text, which makes > > me wonder whether it's the correct image file. It might be, if the same > > file is being reused for different tasks. Minor issue: some of the alt > > texts don't match the image file names.
Do you prefer to add multiple copies of the same image with diff names, or what currently followed is fine ? > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/delete-reset-vpn.xml, line 75 > > <https://reviews.apache.org/r/7106/diff/1/?file=155074#file155074line75> > > > > This alt text doesn't match the .png filename. this is a correct image file. same file has been reused for diff tasks > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/delete-reset-vpn.xml, line 84 > > <https://reviews.apache.org/r/7106/diff/1/?file=155074#file155074line84> > > > > This alt text doesn't match the .png file name. this is a correct image file. same file has been reused for diff tasks > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/release-ip-for-vpc.xml, line 72 > > <https://reviews.apache.org/r/7106/diff/1/?file=155077#file155077line72> > > > > Shouldn't this refer to release-ip-icon.png? this is a correct image file. same file has been reused for diff tasks > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/release-ip-for-vpc.xml, line 75 > > <https://reviews.apache.org/r/7106/diff/1/?file=155077#file155077line75> > > > > This alt text has a typo, an extra "e" at the end. Also doesn't exactly > > match the .png file name. removed. > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/remove-vpc.xml, line 43 > > <https://reviews.apache.org/r/7106/diff/1/?file=155079#file155079line43> > > > > This is correct only if the icon for removing a VPC is the same as the > > icon for destroying an instance. this is a correct image file. same file has been reused for diff tasks. > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/remove-vpc.xml, line 46 > > <https://reviews.apache.org/r/7106/diff/1/?file=155079#file155079line46> > > > > This alt text doesn't match the current .png file name in the imagedata > > tag. this is a correct image file. same file has been reused for diff tasks. > On Sept. 27, 2012, 6:49 a.m., Jessica Tomechak wrote: > > docs/en-US/remove-vpc.xml, line 55 > > <https://reviews.apache.org/r/7106/diff/1/?file=155079#file155079line55> > > > > This alt text doesn't match the .png file name in the imagedata tag. this is a correct image file. same file has been reused for diff tasks. - Radhika ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7106/#review11968 ----------------------------------------------------------- On Sept. 14, 2012, 12:36 p.m., Radhika PC wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7106/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2012, 12:36 p.m.) > > > Review request for cloudstack and David Nalley. > > > Description > ------- > > files for VPC/inter vlan routing > > > Diffs > ----- > > docs/en-US/acquire-new-ip-for-vpc.xml PRE-CREATION > docs/en-US/add-gateway-vpc.xml PRE-CREATION > docs/en-US/add-loadbalancer-rule-vpc.xml PRE-CREATION > docs/en-US/add-portforward-rule-vpc.xml PRE-CREATION > docs/en-US/add-tier.xml PRE-CREATION > docs/en-US/add-vm-to-tier.xml PRE-CREATION > docs/en-US/add-vpc.xml PRE-CREATION > docs/en-US/configure-acl.xml PRE-CREATION > docs/en-US/configure-vpc.xml PRE-CREATION > docs/en-US/delete-reset-vpn.xml PRE-CREATION > docs/en-US/enable-disable-static-nat-vpc.xml PRE-CREATION > docs/en-US/inter-vlan-routing.xml PRE-CREATION > docs/en-US/release-ip-for-vpc.xml PRE-CREATION > docs/en-US/remove-tier.xml PRE-CREATION > docs/en-US/remove-vpc.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/7106/diff/ > > > Testing > ------- > > > Thanks, > > Radhika PC > >
