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.

Sign up for GitHub

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

Jump to bottom

Remove transparency from group images #2346

Open
22 tasks
beutlich opened this issue Oct 9, 2017 · 5 comments
Open
22 tasks

Remove transparency from group images #2346

beutlich opened this issue Oct 9, 2017 · 5 comments
Assignees
@MartinOtter @HansOlsson @ThomasBoedrich
Labels
L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes specification Issue (also) addresses the Modelica language specification task General work that is not related to a bug or feature

Comments

@beutlich
Copy link
Member

beutlich commented Oct 9, 2017 •
edited by tobolar
Loading

When discussing #2344, it was agreed, that the PNG images used as group images should have white background. This refers to the following images in Magnetic.FluxTubes

  • CoaxCylindersEndFaces.png
  • Core_SinglePhase.png
  • Core_ThreePhase1.png
  • CuboidOrthogonalFlux.png
  • CuboidParallelFlux.png
  • EighthOfHollowSphere.png
  • EighthOfSphere.png
  • Eqn_ShapeFunctionsTellinen.png
  • HalfCylinder.png
  • HalfHollowCylinder.png
  • HardMagneticHysteresis.png
  • HollowCylinderAxialFlux.png
  • HollowCylinderRadialFlux.png
  • LeakageAroundPoles.png
  • LeakageWithCoefficient.png
  • MovingCoilActuator_dimensions.png
  • QuarterCylinder.png
  • QuarterHollowCylinder.png
  • QuarterHollowSphere.png
  • QuarterSphere.png
  • SoftMagneticHysteresis1.png
  • Solenoid_dimensions.png
@beutlich beutlich added L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes task General work that is not related to a bug or feature labels Oct 9, 2017
@beutlich beutlich added this to the MSL_next-MINOR-version milestone Oct 9, 2017
@HansOlsson
Copy link
Contributor

I thought the agreement was to first discuss transparency for images and then possibly introduce it consistently, instead of introducing "randomly" combined with other changes.

However, the above seems like existing images - and I don't see a reason to spend time on them before that discussion.

@MartinOtter
Copy link
Member

Yes. First discuss, before making any changes to images

@beutlich
Copy link
Member Author

beutlich commented Oct 9, 2017

Thought we wanted to have white background already now - everywhere.

@HansOlsson
Copy link
Contributor

HansOlsson commented Mar 28, 2018 •
edited by beutlich
Loading

I have added a Modelica specification ticket about this (proposing to standardize on transparent background - that is intended to be white-ish): modelica/ModelicaSpecification#2244

Note that the discussion vs. specification text-ratio is somewhat high for that proposal.

@beutlich beutlich added the specification Issue (also) addresses the Modelica language specification label May 29, 2018
@beutlich beutlich modified the milestones: MSL3.2.3, MSL_next-MAJOR-version May 29, 2018
@beutlich
Copy link
Member Author

beutlich commented May 29, 2018 •
edited
Loading

Rescheduling milestone.

Closed
@beutlich beutlich removed this from the MSL4.0.0 milestone Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MartinOtter MartinOtter

@HansOlsson HansOlsson

@ThomasBoedrich ThomasBoedrich

Labels
L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes specification Issue (also) addresses the Modelica language specification task General work that is not related to a bug or feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@MartinOtter @HansOlsson @beutlich @ThomasBoedrich