Comments (8)
Cheers for looking into this @mchowning I think it might be more the color of the dividers and the padding that is making it look a little funny. Do you mind just checking we're using $primary-ui-05 for the dividers and that the padding is correct as it looks a little tight?
Thanks!
from pocket-casts-ios.
@adamzelinski I had a look and the ThemedDivider
is using the correct color:
pocket-casts-ios/podcasts/Styles.swift
Lines 132 to 137 in 1ed3c5b
It is missing padding though. Here is the current version:
and then with padding of 10 applied:
Is 10 enough, or did you have a different value in mind?
from pocket-casts-ios.
The padding looks good to me thanks @rviljoen however I think the dividers need to be the width of the screen as it looks funny with the padding on either side
from pocket-casts-ios.
The padding looks good to me thanks @rviljoen however I think the dividers need to be the width of the screen as it looks funny with the padding on either side
@adamzelinski Yes, I agree! I have removed the leading and trailing padding for the dividers:
Let me know if this is good to go and I'll submit a PR.
from pocket-casts-ios.
the dividers look really great now thanks @rviljoen however, now all the padding looks off. Can we please make sure all the text elements are in alignment. The padding from both edges should be 16pt.
from pocket-casts-ios.
@adamzelinski Sorry, I missed that! Here's my latest attempt:
In addition to fixing the padding, I also added some space between the Navigation Title ("Edit Folder") and the folder name to match your design.
from pocket-casts-ios.
Awesome, thanks @rviljoen that's looking great happy with that. I also assume that the close icon will be there too correct as it's not showing in your screenshots
from pocket-casts-ios.
@adamzelinski Yes, the close icon is visible in the app, but does not render in the Xcode view preview as it is not included in each individual view. Here is a render from the app on a real device, with the view controller rendering it:
from pocket-casts-ios.
Related Issues (20)
- Add a drop-shadow to the white-background podcasts in the new grids
- Unable to install dependencies to build the project HOT 1
- Add default remote keys for all feature flags HOT 4
- Add Developer Menu option for switching between Production and Staging
- Queries: move it out of Main Thread
- Tweak play button size in new mini player
- Mini-Player dissapears when rotatin on iPad after openning mini-player in full-screen
- Add code to update any pending Auto-Download status depending of streaming download status.
- Downloading streaming episodes with AVExportSession
- New grid UI tweaks HOT 1
- iPad: The episode view shouldn't be cropped when opening it
- Swift 6 and Strict Concurrency Checking
- Add support for Double Tap on WatchOS
- Fix grey shadow in the mini player area (dark themes) HOT 1
- Check the uses of `receive(on: RunLoop.main)` on Publishers
- Downloads: Add Prioritized queue system for Downloads
- A new auto downloaded episode clears the up next queue
- Crash when importing a file after being on Up Next tab
- Transcripts - Phase 1
- Sleep Timer: restarting on the next morning
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from pocket-casts-ios.