Constructive criticism?

Constructive critisism?

This is going to like 200 sales reps and regional offices.

I am by no means a professional...

What do u think?

Seahag

(any help appreciated)

looks good..

but i guess it is really up to the end user to decide...

one thing..

is that us thing after the dollar amounts for US dollars? if so you may want to capitalize it.. also maybe it is just the picture.. but it looks like there is a period after the u but not the s?

Done. :D

I agree with kleinma the U.S. should be capitalized. Other than that I think it looks good.

cool..

thnks.

Quote:

---

Originally posted by SeaHag
cool..

thnks.

---

maybe it is just me... but I wouldn't put the periods there either.... so it would just be US... you may want to get a few other peoples input on that... but for some reason it just doesn't look right to me...

Looks good, change the u.s. to US (no periods). All else looks nice.

If thats US Dollars shouldn't it be after the the amount of money.... I also thought maybe aligning the Sales Price would look better as well. I would put it all in different label boxes to align them like I have them in the picture. But the U.S. is in the same label as the total amount. If you don't already know just put the Cash amount labels to auto size and in code just.

Label1.Caption = "$" & Amount & " U.S."

But looking at your program you are capable of this =) but I shouldn't assume(math teacher from years ago used to say to assume makes an ASS out of U and ME, its probably the only thing I remember from that class :D )

OK..
When u hit calculate things look good..
I will look into CONCATENATE though.. (i hate that word)

US ... I will do..

thnks all

Looks like salesmen... er, salespeople have to type in their
codes... would be good to have both...

a tab and type interface
AND
a point a click interface... if they click on box A1 for Step one,
then A1 shows up below.

just a thought. looks fine.

Ya.. i though of the click interface too.. But... somethings your bound to..

As you type though.. it error handles. only lets you enter
the correct range of values,, then auto tabs to next box..

I can see that causing problems. alot of people like to point and click...

Nitpicking GUI Items:

Alignment at bottom of form.

Use different text than Times New Roman.

You help text should be larger and at the top of the form (most instructions on how to use, in my opinion, should be at the top...)

Possibly different color than standard grey...

Will do.. makes sense..
:)

Seahag

My personal preference would be to implement a point and click interface and also to dispense with the "calculate" button so that it automatically calculates the value whenever all selections are made.

I guess it depends how much time you have been allocated to do it though.

I thought about it..
But you need all the info to make the calculation.. i bant think a way around it.. too many varaibles


Any suggestions ...

Well all the info is only going to be 7 variables isn't it. One way of doing it would be to have the 7 text boxes as an array and then in the onchange event check they are all full and calculate accordingly

You could use an array to store those variables and calculate... Just loop each part...

Looks good :)

I see... ..

I guess its too late for that.. Would be all that hard come to think of it..
Thnks..

Quote:

---

Originally posted by James Stanich
You could use an array to store those variables and calculate... Just loop each part...

---

Ooo.. lots of arrays...
I did this a year ago.. and now its been rehached.
This was my first attempt at anything big..
eeekk ..




Thanks jesus4u

:)