adrian Posted June 9 Share Posted June 9 40 minutes ago, ryan said: Ah okay, I see it now. Adding to our list of updates. Actually, on this note, why do RM fields use buttons, but repeater fields don't? Link to comment Share on other sites More sharing options...
ryan Posted June 9 Author Share Posted June 9 Quote I've honestly never looked at that option - it seems like a lot of work though on a site with a lot of fields. As I mentioned in my post about this, I thought the "Input types that should be offset with additional top/bottom margin." via Modules > Core > AdminThemeUikit would be the place to set this, but it doesn't seem to do anything. Even if it did, it's still a decent amount of work to set all types. That's the way I use it at least. And it's not much work because it's something usually done while setting up a field, so it's just part of the field creation process. But if you are going back to add it onto lots of existing fields, then maybe that might seem more laborious, as it would be for any field setting, but you'd only have to do it once. In addition, it's not something you'd want to do for every field. I find it useful for certain fieldsets, repeaters, etc. It's kind of a case-by-case basis. I'd forgotten about that setting in AdminThemeUikit where you can select it by Fieldtype. It appears that does not work in the new theme, so I'll have to look into that. Quote Actually, on this note, why do RM fields use buttons, but repeater fields don't? Since adding a matrix field involves selecting a type, it's a different kind of input. But there shouldn't be a button on the matrix type at all, so it's most likely a new theme bug to figure out. Link to comment Share on other sites More sharing options...
Mikel Posted June 10 Share Posted June 10 Wenn using the admin theme on touch devices (iOS) the visual aid of the highlighted background is missing. This is kind of irritating, especially when pages are hidden, because the page title does not get highlighted either. Link to comment Share on other sites More sharing options...
Mikel Posted June 10 Share Posted June 10 We just discovered something funny: The design of the column width slider handle is from a galaxy far, far away, isn´t it? 😉 1 3 Link to comment Share on other sites More sharing options...
elabx Posted June 10 Share Posted June 10 6 hours ago, Mikel said: The design of the column width slider handle is from a galaxy far, far away, isn´t it? 😉 Skeumorphism is so back! Ask Airbnb crew! /j 1 Link to comment Share on other sites More sharing options...
szabesz Posted June 10 Share Posted June 10 4 hours ago, elabx said: Skeumorphism is so back! Back or not, having only one lone icon/image among non-skeuomorph icons/images is, at the very least, questionable. 1 Link to comment Share on other sites More sharing options...
BrendonKoz Posted June 10 Share Posted June 10 3 hours ago, szabesz said: Back or not, having only one lone icon/image among non-skeuomorph icons/images is, at the very least, questionable. It's a holdout from the original theme, it likely was overlooked in the new. Probably should be removed in the base theme (imho) realistically, though that's a matter of opinion. I'd imagine it would potentially be easier to style it if it were a browser-native control instead of a range slider that was created prior to browsers having native interfaces for it. This CSS might clean it up a bit. The new theme could likely use some CSS variables in place of the colors below, I simply targeted the base CSS instead. .ui-slider .ui-slider-handle { border-width:0; outline:none; border:none; width:17px; height:17px; background-color:transparent; background:currentColor 0 0 no-repeat; border-radius:21px; box-shadow: 0 2px 2px gray; } @ryan If we're using the admin.css override, and users are allowed to choose their admin theme, is there currently a way to target our overrides to the sub-themes/styles individually? I see the main theme name is appended to the body classes, but not the sub-theme name, and since we can only have one override file, currently... Is there a way to handle that scenario that I'm not seeing? I'm also wondering if there might be opportunity for others to provide sub-themes to the AdminStyleUiKit, and if so, perhaps the naming of the options should be something other than "Original" and "Default"? Default is typically a word assigned to what would be chosen for someone, not a choice to be made. (I do understand the reasoning, though.) I'd be quite happy if Konkat chose a name for their theme/style, whether it just be Konkat, 2025, some iteration therein, or anything their hearts desire! If a body class were added for subthemes, it'd be weird seeing "Original" or "Default". 😉 - - - Installed the DEV branch. The dark mode looks more like a high contrast mode to me. I'm not yet sure what suggestions I have for it, but compared to the light mode, it's quite jarring. (For what it's worth, I use dark mode by default in systems that offer it.) I'll try to think on that a bit more. 2 Link to comment Share on other sites More sharing options...
Mikel Posted June 12 Share Posted June 12 From the start, I asked myself why I was somehow not completely satisfied with the display of the checkboxes as switches: We actually use switches quite heavily in some of our webapps, so I looked at the examples and therefore suggest making the “inactive” status brighter. This makes it easier to recognize the status of the checkbox. Below is an example of iOS switches. I also find it problematic when the theme color is red: To me, this makes the checkbox appear “false” even though it is “true”. Making the “unchecked” state lighter in color, also would help in this case. Below is an example of one of our web apps. There we chose to give both states a color: That makes it easy to understand the state of the toggle (checkbox). 2 Link to comment Share on other sites More sharing options...
bernhard Posted June 12 Share Posted June 12 2 hours ago, Mikel said: That makes it easy to understand the state of the toggle (checkbox). As long as you are not color blind I guess 🙂 Link to comment Share on other sites More sharing options...
Pete Posted June 12 Share Posted June 12 Re. toggles I think the iOS way of doing toggles is the most understandable as lack of colour is "inactive" in this case and one colour for the "on" state - preferable not red, or if you do choose red as your primary theme colour at least a simple way to override that in CSS. I've not really looked at the new theme yet but do primary action buttons still have a separate class to secondary? I was quite used to overriding a bunch of stuff on the current admin theme - probably more than was intended - via admin.less so I'll have to take a peek at some point. Link to comment Share on other sites More sharing options...
Mikel Posted June 12 Share Posted June 12 11 hours ago, bernhard said: As long as you are not color blind I guess 🙂 😄 Yes, that would be not so nice. But we created the app (and the toggles) for a small user group with no color blind members and the mission was to visualize the states of many projects and subtasks at one glance. The green/red scheme achieved the best results in our test settings, so we went with that. 😉👍 But have a look for yourself: https://frameless.at/en/case-studies/real-estate-marketing/ 1 Link to comment Share on other sites More sharing options...
bernhard Posted June 13 Share Posted June 13 Using the new theme on mobile seems very hard to me at the moment. The menu button is not available and the page tree with all pagelist actions is... somewhat strange. pagelist actions are sometimes not visible at all, sometimes they appear when clicked, sometimes they don't. Link to comment Share on other sites More sharing options...
stehlo Posted June 14 Share Posted June 14 (edited) REMOVE Edited Friday at 08:39 PM by stehlo 1 Link to comment Share on other sites More sharing options...
ryan Posted June 14 Author Share Posted June 14 @stehlo Quote Releasing a grossly unfinished and cross-browser incompatible subproduct, completely neglecting minority browsers, as a default version is not the best idea. Ryan's original themes do not make this mistake. We are beta testing this admin design on the dev branch, nothing has been released in an official version. If you feel strongly about the new design, you may prefer to stick with the main/master branch while we continue testing and optimizing the new design. I have no idea what you are talking about with regard to being unfinished or cross-browser incompatible, so if you are running into specific issues it would help if you open a GitHub issue report where you can elaborate on what it is you are observing as being unfinished or cross-browser incompatible. Quote Running after numbers whilst going against the current user base – that is a trap, which many have fallen into. I've mentioned this a few times before, but we are not replacing the Original theme, we are adding another styling option for it. If you prefer the original, you should keep using it. Understand that not everyone has the same preferences here. I prefer the new design, it sounds like you prefer the original. We are not taking anything away from the current user base. Providing more options to appeal to a broader audience is not a trap. Quote Using the new theme on mobile seems very hard to me at the moment. The menu button is not available and the page tree with all pagelist actions is... somewhat strange. pagelist actions are sometimes not visible at all, sometimes they appear when clicked, sometimes they don't. @bernhard I can't fully duplicate what you've described, using current Chrome on Android 15 (Pixel). Can you provide more details on what device you are using? I am seeing the larger margins between some pages though, it appears for any longer page titles that wrap when the page actions appear, so I can duplicate that part at least. But everything still works as expected, menus, etc., so need more detail on the parts that aren't working there. Link to comment Share on other sites More sharing options...
diogo Posted June 14 Share Posted June 14 11 hours ago, stehlo said: @jploch @diogo @ryan Running after numbers whilst going against the current user base – that is a trap, which many have fallen into. Masses are not making sane decisions; therefore, sacrificing the unique quality aspects of a niche product by trying to appeal to "others" or "all" tends to lead to "neither" and "none." Releasing a grossly unfinished and cross-browser incompatible subproduct, completely neglecting minority browsers, as a default version is not the best idea. Ryan's original themes do not make this mistake. Tagging three people on your post must mean you really wanted to be heard. Why such an unhelpful post, then? If you want to vent your frustration, at least put a minimum effort on it. 1 1 Link to comment Share on other sites More sharing options...
AndZyk Posted June 14 Share Posted June 14 (edited) Captured on iPhone 8 Safari (yes my phone is old) The navigation has issues and the page tree could look more compact. 😉 Edited June 14 by AndZyk 2 Link to comment Share on other sites More sharing options...
bernhard Posted June 15 Share Posted June 15 I'm using a regular UIkit modal... Original theme: With the default theme the modal is vertically centered, which causes a weird glitch when closing it: As the UIkit docs mention there is a standardised way to center the modal: https://getuikit.com/docs/modal#center-modal. And as you can see there it will work properly if used properly. I don't know why the default modal is vertically centered with the new theme, but I know that adding these kind of visual "optimisations" causes problems. Could you please stick to default UIkit markup and concepts as much as possible with your overrides to avoid these kind of issues? Thank you. 2 2 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now