X Tutup
Skip to content

Smaller ownCloud logo in mail image#39501

Merged
phil-davis merged 1 commit intomasterfrom
smaller-mail-logo
Nov 17, 2021
Merged

Smaller ownCloud logo in mail image#39501
phil-davis merged 1 commit intomasterfrom
smaller-mail-logo

Conversation

@ChrisEdS
Copy link

@ChrisEdS ChrisEdS commented Nov 17, 2021

Description

Changing the size of the ownCloud mail logo

Motivation and Context

The logo looks a bit too big in the mails, so I reduced the size by about 50%.

Screenshots (if appropriate):

Before:
image

After:
image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@ChrisEdS ChrisEdS self-assigned this Nov 17, 2021
@update-docs
Copy link

update-docs bot commented Nov 17, 2021

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@AlexAndBear
Copy link

AlexAndBear commented Nov 17, 2021

Changelog item needed - LGTM then

@ownclouders
Copy link
Contributor

ownclouders commented Nov 17, 2021

💥 Acceptance tests pipeline apiComments-mariadb10.2-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/core/33753/51/1

@ownclouders
Copy link
Contributor

ownclouders commented Nov 17, 2021

💥 Acceptance tests pipeline apiShareReshareToShares2-mariadb10.2-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/core/33753/76/1

@ownclouders
Copy link
Contributor

ownclouders commented Nov 17, 2021

💥 Acceptance tests pipeline apiShareCreateSpecialToShares2-mariadb10.2-php7.4 failed. The build has been cancelled.

https://drone.owncloud.com/owncloud/core/33753/61/1

@phil-davis
Copy link
Contributor

phil-davis commented Nov 17, 2021

One of the drone CI pipelines had a timeout with slow-loading docker images. I restarted CI.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 17, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis phil-davis merged commit 5990a04 into master Nov 17, 2021
@delete-merged-branch delete-merged-branch bot deleted the smaller-mail-logo branch November 17, 2021 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JammingBen JammingBen Awaiting requested review from JammingBen

1 more reviewer

@AlexAndBear AlexAndBear AlexAndBear approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ChrisEdS ChrisEdS

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ChrisEdS @AlexAndBear @ownclouders @phil-davis @JammingBen
X Tutup