Skip to content

[ticket/15605] Correct Right-Margin for ribbon image#5183

Merged
Nicofuma merged 5 commits intophpbb:masterfrom
dnssh:phpbb-dev
Jul 8, 2018
Merged

[ticket/15605] Correct Right-Margin for ribbon image#5183
Nicofuma merged 5 commits intophpbb:masterfrom
dnssh:phpbb-dev

Conversation

@dnssh
Copy link
Copy Markdown
Contributor

@dnssh dnssh commented Mar 29, 2018

Checklist:

  • Correct branch: master for new features; 3.2.x for fixes
  • Tests pass
  • Code follows coding guidelines: master and 3.2.x
  • Commit follows commit message format

Tracker ticket (set the ticket ID to 15605):

https://tracker.phpbb.com/browse/PHPBB3-15605

@hanakin
Copy link
Copy Markdown
Member

hanakin commented Mar 29, 2018

@dnssh we have very strict coding guidelines we are implementing for version 3.3 related to CSS we are still working on all the documentation but we are using stylelint for linting rules. have a look at the Travis results https://travis-ci.org/phpbb/phpbb/jobs/359943288#L502 if you have any questions please feel free to ask here

@hanakin
Copy link
Copy Markdown
Member

hanakin commented Mar 30, 2018

Check the commit I left on your commit as the message needs fixed slightly

@dnssh
Copy link
Copy Markdown
Contributor Author

dnssh commented Mar 30, 2018

Hello Sir,
@hanakin Thank you for your kind suggestion.
Pls review whether below commit has correct format

$ git commit -m "[ticket/15605] Correct Right-Margin for ribbon image

PHPBB3-15605"

Regards,
Devansh Shah

@hanakin
Copy link
Copy Markdown
Member

hanakin commented Mar 30, 2018

@dnssh looks good

@dnssh
Copy link
Copy Markdown
Contributor Author

dnssh commented Mar 31, 2018

Thank You @hanakin ,all the changes have been committed successfully.

@hanakin
Copy link
Copy Markdown
Member

hanakin commented Mar 31, 2018

@dnssh ok a couple of things. First, why did you create a new ticket in the tracker? Please do not do this use the ticket that is there next time. second, all you need is a margin: 5px; auto does nothing when positioning absolutely so not sure why that's there and left and bottom have no effect so might as well simplify the amount of code.

@dnssh
Copy link
Copy Markdown
Contributor Author

dnssh commented Mar 31, 2018

Ok thanks ,will consider these things further.

@hanakin
Copy link
Copy Markdown
Member

hanakin commented Apr 5, 2018

@dnssh are you going to make the suggested change?

@dnssh
Copy link
Copy Markdown
Contributor Author

dnssh commented Apr 6, 2018

OK Sir @hanakin made those suggested changes

Copy link
Copy Markdown
Member

@hanakin hanakin left a comment

Choose a reason for hiding this comment

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

I Approve

@Nicofuma Nicofuma added this to the 3.3.0-a1 milestone Jul 8, 2018
@Nicofuma Nicofuma merged commit af077a9 into phpbb:master Jul 8, 2018
Nicofuma added a commit that referenced this pull request Jul 8, 2018
[ticket/15605] Correct Right-Margin for ribbon image

* github.com:/phpbb/phpbb:
  [ticket/15605] Correct Right Margin for ribbon image
  [ticket/15605] Correct Right-Margin for ribbon image 4
  [ticket/15605] Correct Right-Margin for ribbon image 3
  [ticket/15605] Updated Correct Right-Margin for ribbon image 2
  [ticket/15605] Correct Right-Margin for ribbon image 1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants