It looks fine to me, the only things I'd suggest altering are simply to make it easier to rember the order of things, or rather, place them in a more logical order.
But in the end it'll function the same anyway.
Example:
Table Brokers:
BrokerID
OfficeID
FirstName
LastName
Title
DirectPhone
MobilePhone
Email
Fax
Table Clients:
ClientID
ClientPassword
Email
FirstName
LastName
Phone
Title I would put before FirstName.
Simply because when we call up that data inside SQL itself the way you have it we'll get something like:
10001 100 John Doe Mr. 1234563456... etc. Email and
Phone I'd personally put these two next to eachother, since they're both contact information.
Once again, totally unnecissary, but I like to keep things organized.
So long story short, your layout seems fine.