Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Macro renamed and modified, chapter12 renamed as IndividualSpreadFooting, Run Script Added #15

Merged
merged 6 commits into from
Jan 6, 2016

Conversation

manpreet9112
Copy link
Contributor

No description provided.

@@ -18,10 +18,10 @@ \section{Introduction} Clause 34 of the \citetitle{IS2000} gives the

\item 25\% excess pressure on edge of footing was allowed by the
\citetitle{IS1964} when a footing is eccentrically loaded. This
concession is withdrawn by the \citetitle{IS2000}, thereby adding to
concession is withdrawn by the ,\citetitle{IS2000} thereby adding to
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why there is 'space' between 'the' and comma?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unintentionally it happens. I will remove this extra space in next PR .

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is right? Do you need comma in this line?
If yes, where do you want to place it?

Let me know. I can merge after correcting this.

Which one of the following do you need? (I guess first one is right)

  1. withdrawn by the, \citetitle{IS2000} thereby
  2. withdrawn by the,\citetitle{IS2000} thereby

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be like this

concession is withdrawn by the \citetitle{IS2000}, thereby adding to

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now I made changes as you said "we don't need this comma". :(
You should add new comment, as editing comment don't push notifications.

@mandeeps708 mandeeps708 merged commit 7da4a99 into GreatDevelopers:master Jan 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants