I did not suggest moving the "root" of desktop, only tha alias at most. Unless there was a reason for keeping the alias in the patch file (like some clients in need of it?) I'd say we remove it no later than 1.0.2 and stick to "genericDesktop". WDYT?
Almost every item in the main data file is "*generic", and as you say people can still add one with their own name in the patch file. Even the "patchDevice" which is an empty shell seems no more than a stub. If we keep that, I suggest we put a little bit of XML documentation there to explain how people can extend device signatures there. For e.g. the 600x800 screen a custom extension makes sense, but there's no reason not to keep a default signature in the other file. We don't need to follow WURFL's path where we can do better, the performance of the W3C implementation already showed ODDR/DeviceMap does a lot better there;-) Werner On Wed, Sep 10, 2014 at 1:57 PM, Reza Naghibi < [email protected]> wrote: > Remember, any core device or pattern can be changed by defining the Id in > the patch file and then overriding the value. > > I don't think that any of our definitions should live in the patchfile. > Wurfl kept desktops in the patchfile because it did a poor job of detecting > them. That's not the case for us which is why we expanded the core > definitions to bots and other device classes. > > Patching should only be required to add functionality. If we are missing > functionality... we should consider fixing it in the core definitions. > > So I'm for keeping all device and pattern definitions in the core files. > > > > <div>-------- Original message --------</div><div>From: Werner Keil < > [email protected]> </div><div>Date:09/10/2014 6:27 AM (GMT-05:00) > </div><div>To: [email protected] </div><div>Subject: Re: > Data </div><div> > </div>It would be great to allow adjustment in the "Patch" file by moving a > device definition like "desktopDevice" there. > Whether it's named "desktopDevice" or "patchDesktop..." or similar does not > really matter. > > It seems not show-stopper for 1.0.1, as long as the W3C compliant builder > signatures work now, I would not insist on changing it now, but we may want > to clean that up in the next release. > > Werner > > On Wed, Sep 10, 2014 at 1:35 AM, eberhard speer jr. <[email protected]> > wrote: > > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA1 > > > > RIM - BlackBird...got it, no change required. Easy. > > > > And iOS and iPhone OS ? similar case ? > > > > esjr > > -----BEGIN PGP SIGNATURE----- > > Version: GnuPG v2.0.22 (MingW32) > > > > iQEcBAEBAgAGBQJUD465AAoJEOxywXcFLKYcgFEH/0SFzkl3h8WefhOZWM7tIAIj > > SawluAF1iBugkaP+G62zafK2h7OS0ihjRnic/+lcvxIxzlUkDQnt1yy2OZqDeIhm > > jS7tM0VAfkykHcvgB1yolRxUSRtAT/rGdAeUnIzojqqR7C4+FUXE2anVvwFEcGqz > > 27UAX/vRkq3Pq89ZBLcu7CTMJEsbzMFxQvL9Uelf4JWyWSeKemlV2Bc/KKa8j9Uc > > s2ucwS2KFY5tFYK1QllhUEhQEOgVNKxi8K8DLoc+KI00b5w3HMBif46ZJ/Yw4vpC > > b1duhGZToVcbku7EgbhcjjDYxtSiO5ROzhm6iT2zvgWo93ZXWm0yHpbDYhZs47E= > > =d7Ls > > -----END PGP SIGNATURE----- > > >
